Changeset 38460

Timestamp:

08/31/2016 06:05:06 AM (8 years ago)

Author:

dd32

Message:

Bootstrap: Check that ini_get_all() exists before calling it, allows us to work around hosts who disable the function for "security purposes".

Merges [38431] to the 4.6 branch.
Fixes #37680.

Location:

branches/4.6

Files:

• . (modified) (1 prop)
• src/wp-includes/load.php (modified) (2 diffs)

branches/4.6/src/wp-includes/load.php

@@ -1018 +1018 @@
 
     if ( ! isset( $ini_all ) ) {
-        $ini_all = ini_get_all();
-    }
+        $ini_all = false;
+        // Sometimes `ini_get_all()` is disabled via the `disable_functions` option for "security purposes".
+        if ( function_exists( 'ini_get_all' ) ) {
+            $ini_all = ini_get_all();
+        }
+    }
 
     // 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.
@@ -1026 +1030 @@
     }
 
+    // If we were unable to retrieve the details, fail gracefully to assume it's changeable.
+    if ( ! is_array( $ini_all ) ) {
+        return true;
+    }
+
     return false;
 }