Changeset 38017 for trunk/src/wp-includes/load.php

Timestamp:

07/08/2016 06:35:22 PM (8 years ago)

Author:

ocean90

Message:

Bootstrap: Make wp_is_ini_value_changeable() compatible with PHP 5.2.6 - 5.2.17.

There is a bug in PHP 5.2.6 - 5.2.17 (​https://bugs.php.net/bug.php?id=44936, ​https://3v4l.org/IL0A2) which changes the access level of a setting to 63 after ini_set() was called.
To continue comparing the access value against INI_ALL and INI_USER use the bit operator & 7:

• 1 & 7 === 1 (INI_USER)
• 2 & 7 === 2 (INI_PERDIR)
• 4 & 7 === 4 (INI_SYSTEM)
• 7 & 7 === 7 (INI_ALL)
• 63 & 7 === 7 (INI_ALL)

See [38015].
See #32075.

File:

• trunk/src/wp-includes/load.php (modified) (1 diff)

trunk/src/wp-includes/load.php

@@ -1021 +1021 @@
     }
 
-    if ( isset( $ini_all[ $setting ]['access'] ) && ( INI_ALL === $ini_all[ $setting ]['access'] || INI_USER === $ini_all[ $setting ]['access'] ) ) {
+    // Bit operator to workaround https://bugs.php.net/bug.php?id=44936 which changes access level to 63 in PHP 5.2.6 - 5.2.17.
+    if ( isset( $ini_all[ $setting ]['access'] ) && ( INI_ALL === ( $ini_all[ $setting ]['access'] & 7 ) || INI_USER === ( $ini_all[ $setting ]['access'] & 7 ) ) ) {
         return true;
     }