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

Timestamp:

07/18/2016 07:42:48 PM (8 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();