resubmitting last Monday updates and adding php 5.1.0 timezone support

[squirrelmail.git] / src / configtest.php
diff --git a/src/configtest.php b/src/configtest.php

index 7377d0c4eeb12c6627a59b2f5949c450ad9b7f9f..1ffb18d2f418b95e801ea78964b650654641276a 100644 (file)
--- a/src/configtest.php
+++ b/src/configtest.php
@@ -52,6 +52,11 @@ if (file_exists(SM_PATH . 'config/config.php')) {
  require(SM_PATH . 'functions/global.php');
  require(SM_PATH . 'functions/strings.php');
  
+/**
+ * get_location starts session and must be run before output is started.
+ */
+$test_location = get_location();
+
  ?><!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
    "http://www.w3.org/TR/1999/REC-html401-19991224/loose.dtd">
  <html>
@@ -111,7 +116,9 @@ if(!check_php_version(4,1,0)) {
  }
  
  echo $IND . 'PHP version ' . PHP_VERSION . ' OK. (You have: ' . phpversion() . ". Minimum: 4.1.0)<br />\n";
-if ((bool) ini_get('register_globals')) {
+/* test for boolean false and any string that is not equal to 'off' */
+if ((bool) ini_get('register_globals') && 
+    strtolower(ini_get('register_globals'))!='off') {
         do_err('You have register_globals turned on.  This is not an error, but it CAN be a security hazard.  Consider turning register_globals off.', false);
  }
  $php_exts = array('session','pcre');
@@ -237,7 +244,32 @@ if (isset($plugins[0])) {
              do_err('You have enabled the <i>'.$plugin.'</i> plugin, which causes problems with this version of SquirrelMail. Please check the ReleaseNotes or other documentation for more information.', false);
          }
      }
-    echo $IND . "Plugins OK.<br />\n";
+    // load plugin functions
+    include_once(SM_PATH . 'functions/plugin.php');
+    // turn on output buffering in order to prevent output of new lines
+    ob_start();
+    foreach ($plugins as $name) {
+        use_plugin($name);
+    }
+    // get output and remove whitespace
+    $output = trim(ob_get_contents());
+    ob_end_clean();
+    // if plugins output more than newlines and spacing, stop script execution.
+    if (!empty($output)) {
+        $plugin_load_error = 'Some output is produced when plugins are loaded.'
+            .' Usually it means error. Output said: '.htmlspecialchars($output);
+        do_err($plugin_load_error);
+    }
+    /**
+     * Hook is added in 1.5.2. Plugins should print error message and return true
+     * if there is an error in plugin.
+     */
+    $plugin_err = boolean_hook_function('configtest');
+    if($plugin_err) {
+        do_err('Some plugin tests failed.');
+    } else {
+        echo $IND . "Plugins OK.<br />\n";
+    }
  } else {
      echo $IND . "Plugins are not enabled in config.<br />\n";
  }
@@ -269,7 +301,7 @@ if ( $squirrelmail_default_language != 'en_US' ) {
      echo $IND . "Default language OK.<br />\n";
  }
  
-echo $IND . "Base URL detected as: <tt>" . htmlspecialchars(get_location()) . "</tt><br />\n";
+echo $IND . "Base URL detected as: <tt>" . htmlspecialchars($test_location) . "</tt><br />\n";
  
  /* check minimal requirements for other security options */
  
@@ -483,7 +515,7 @@ if (function_exists('gettext')) {
  
      /* optional setlocale() tests. Should work only on glibc systems. */
      if (sqgetGlobalVar('testlocales',$testlocales,SQ_GET)) {
-        include_once(SM_PATH . 'functions/i18n.php');
+        include_once(SM_PATH . 'include/languages.php');
          echo $IND . $IND . 'Testing translations:<br>';
          foreach ($languages as $lang_code => $lang_data) {
              /* don't test aliases */
@@ -556,9 +588,9 @@ if (function_exists('iconv')) {
  } else {
      echo "Iconv functions are unavailable.<br />\n";
  }
-// same test as in include/validate.php
+// same test as in include/init.php + date_default_timezone_set check
  echo "$IND timezone - ";
-if ( (!ini_get('safe_mode')) ||
+if ( (!ini_get('safe_mode')) || function_exists('date_default_timezone_set') ||
          !strcmp(ini_get('safe_mode_allowed_env_vars'),'') ||
          preg_match('/^([\w_]+,)*TZ/', ini_get('safe_mode_allowed_env_vars')) ) {
      echo "Webmail users can change their time zone settings. \n";