From: Brion Vibber <brion@status.net>
Date: Tue, 12 Jan 2010 15:24:43 +0000 (-0800)
Subject: Ticket 2152: avoid fatal when php.ini disables dl via disabled_functions (function_ex... 
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=cbc3c7b141b8cfd8baa35a580e9fc9ae4f50343c;p=quix0rs-gnu-social.git

Ticket 2152: avoid fatal when php.ini disables dl via disabled_functions (function_exists returns false even though it exists and cannot be redefined)
---

diff --git a/lib/common.php b/lib/common.php
index 7342c177a6..b280afec02 100644
--- a/lib/common.php
+++ b/lib/common.php
@@ -45,11 +45,20 @@ define('FOREIGN_FRIEND_RECV', 2);
 
 set_include_path(get_include_path() . PATH_SEPARATOR . INSTALLDIR . '/extlib/');
 
-# To protect against upstream libraries which haven't updated
-# for PHP 5.3 where dl() function may not be present...
+// To protect against upstream libraries which haven't updated
+// for PHP 5.3 where dl() function may not be present...
 if (!function_exists('dl')) {
-    function dl($library) {
-        return false;
+    // function_exists() returns false for things in disable_functions,
+    // but they still exist and we'll die if we try to redefine them.
+    //
+    // Fortunately trying to call the disabled one will only trigger
+    // a warning, not a fatal, so it's safe to leave it for our case.
+    // Callers will be suppressing warnings anyway.
+    $disabled = array_filter(array_map('trim', explode(',', ini_get('disable_functions'))));
+    if (!in_array('dl', $disabled)) {
+        function dl($library) {
+            return false;
+        }
     }
 }