From 78789beda9d71bb7fcf725839ad72f17ce9ff115 Mon Sep 17 00:00:00 2001 From: Dominik Schilling Date: Fri, 8 Jul 2016 18:35:22 +0000 Subject: [PATCH] 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. git-svn-id: https://develop.svn.wordpress.org/trunk@38017 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/load.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/wp-includes/load.php b/src/wp-includes/load.php index 3b79351b97..11c36fda97 100644 --- a/src/wp-includes/load.php +++ b/src/wp-includes/load.php @@ -1020,7 +1020,8 @@ function wp_is_ini_value_changeable( $setting ) { $ini_all = ini_get_all(); } - 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; }