Changeset 26449 for trunk/src/wp-includes/functions.php

Timestamp:

11/28/2013 02:19:24 AM (11 years ago)

Author:

dd32

Message:

Fix a regression in wp_mkdir_p() where the $mode of the parent folder is not correctly applied to all created paths. Fixes #25822 for trunk

File:

• trunk/src/wp-includes/functions.php (modified) (2 diffs)

trunk/src/wp-includes/functions.php

@@ -1384 +1384 @@
 
     // Get the permission bits.
-    if ( $target_parent && '.' != $target_parent ) {
-        $stat = @stat( $target_parent );
+    $dir_perms = false;
+    if ( $stat = @stat( $target_parent ) ) {
         $dir_perms = $stat['mode'] & 0007777;
     } else {
@@ -1392 +1392 @@
 
     if ( @mkdir( $target, $dir_perms, true ) ) {
+
+        // If a umask is set that modifies $dir_perms, we'll have to re-set the $dir_perms correctly with chmod()
+        if ( $dir_perms != $dir_perms & ~umask() ) {
+            $folder_parts = explode( '/', substr( $target, strlen( $target_parent ) + 1 ) );
+            for ( $i = 1; $i <= count( $folder_parts ); $i++ ) {
+                @chmod( $target_parent . '/' . implode( '/', array_slice( $folder_parts, 0, $i ) ), $dir_perms );
+            }
+        }
+
         return true;
     }