Changeset 38094 for trunk/src/wp-admin/includes/class-wp-filesystem-ftpext.php

Timestamp:

07/18/2016 07:42:48 PM (10 years ago)

Author:

ocean90

Message:

Filesystem API: Cleanup temporary file when the temporary file couldn't be opened.

Props ruud@joyo.
See #34772.
Fixes #36942, #36943.

File:

• trunk/src/wp-admin/includes/class-wp-filesystem-ftpext.php (modified) (3 diffs)

trunk/src/wp-admin/includes/class-wp-filesystem-ftpext.php

@@ -121 +121 @@
                         return false;
                 }
-                
+
                 if ( ! @ftp_fget( $this->link, $temp, $file, FTP_BINARY ) ) {
                         fclose( $temp );
@@ -127 +127 @@
                         return false;
                 }
-                
+
                 fseek( $temp, 0 ); // Skip back to the start of the file being written to
                 $contents = '';
@@ -160 +160 @@
                 $tempfile = wp_tempnam($file);
                 $temp = fopen( $tempfile, 'wb+' );
-                if ( ! $temp )
-                        return false;
+
+                if ( ! $temp ) {
+                        unlink( $tempfile );
+                        return false;
+                }
 
                 mbstring_binary_safe_encoding();