From 0b6d022e6fa9dc08ddbc8ba420cdd58a690cf852 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak Date: Mon, 4 Jun 2012 16:05:57 +0200 Subject: Fix magic_quotes_sybase handling (#1488506) Improved handling of critical PHP settings - try to set them to required value before die() --- program/include/iniset.php | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) (limited to 'program/include/iniset.php') diff --git a/program/include/iniset.php b/program/include/iniset.php index 2961beed6..a2f43b380 100644 --- a/program/include/iniset.php +++ b/program/include/iniset.php @@ -20,18 +20,20 @@ +-----------------------------------------------------------------------+ */ -// Some users are not using Installer, so we'll check some -// critical PHP settings here. Only these, which doesn't provide -// an error/warning in the logs later. See (#1486307). -$crit_opts = array( - 'mbstring.func_overload' => 0, +$config = array( + 'error_reporting' => E_ALL &~ (E_NOTICE | E_STRICT), + // Some users are not using Installer, so we'll check some + // critical PHP settings here. Only these, which doesn't provide + // an error/warning in the logs later. See (#1486307). + 'mbstring.func_overload' => 0, 'suhosin.session.encrypt' => 0, - 'session.auto_start' => 0, - 'file_uploads' => 1, - 'magic_quotes_runtime' => 0, + 'session.auto_start' => 0, + 'file_uploads' => 1, + 'magic_quotes_runtime' => 0, + 'magic_quotes_sybase' => 0, // #1488506 ); -foreach ($crit_opts as $optname => $optval) { - if ($optval != ini_get($optname)) { +foreach ($config as $optname => $optval) { + if ($optval != ini_get($optname) && @ini_set($optname, $optval) === false) { die("ERROR: Wrong '$optname' option value. Read REQUIREMENTS section in INSTALL file or use Roundcube Installer, please!"); } } @@ -59,8 +61,6 @@ if (set_include_path($include_path) === false) { die("Fatal error: ini_set/set_include_path does not work."); } -ini_set('error_reporting', E_ALL &~ (E_NOTICE | E_STRICT)); - // increase maximum execution time for php scripts // (does not work in safe mode) @set_time_limit(120); -- cgit v1.2.3