Переглянути джерело

rg=0 broke some functionality that plugins relied on. fix 682962

Thijs Kinkhorst 22 роки тому
батько
коміт
992e6debba
3 змінених файлів з 10 додано та 17 видалено
  1. 2 0
      ChangeLog
  2. 2 2
      src/login.php
  3. 6 15
      src/redirect.php

+ 2 - 0
ChangeLog

@@ -15,6 +15,8 @@ Version 1.4.0 CVS
   - Updates in conf.pl to infamous delete_folder setting, including
     addition of appropriate default value for courier and UW.
   - Fix for date/time display in certain timezones
+  - Fix some features of login.php that are used by some plugins and was broken
+    by register_globals = off.
   
 Version 1.4.0 RC 2a
 -------------------

+ 2 - 2
src/login.php

@@ -43,7 +43,7 @@ $base_uri = sqm_baseuri();
  * In case the last session was not terminated properly, make sure
  * we get a new one.
  */
- 
+
 sqsession_destroy();
  
 header('Pragma: no-cache');
@@ -69,7 +69,7 @@ $username_form_name = 'login_username';
 $password_form_name = 'secretkey';
 do_hook('login_top');
 
-$loginname_value = (isset($loginname) ? htmlspecialchars($loginname) : '');
+$loginname_value = (sqGetGlobalVar('loginname', $loginname) ? htmlspecialchars($loginname) : '');
 
 /* If they don't have a logo, don't bother.. */
 if (isset($org_logo) && $org_logo) {

+ 6 - 15
src/redirect.php

@@ -53,22 +53,13 @@ sqsession_unregister ('user_is_logged_in');
 sqsession_register ($base_uri, 'base_uri');
 
 /* get globals we me need */
-if (isset($_POST['login_username'])) {
-    $login_username = $_POST['login_username'];
-}
-if (!isset($_COOKIE['squirrelmail_language']) ||
-    $squirrelmail_language == '' ) {
-    $squirrelmail_language = $squirrelmail_default_language;
-}
-else {
-    $squirrelmail_language = $_COOKIE['squirrelmail_language'];
-}
-if (isset($_POST['secretkey'])) {
-    $secretkey = $_POST['secretkey'];
-}
-if (isset($_POST['js_autodetect_results'])) {
-    $js_autodetect_results = $_POST['js_autodetect_results'];
+sqGetGlobalVar('login_username', $login_username);
+sqGetGlobalVar('secretkey', $secretkey);
+sqGetGlobalVar('js_autodetect_results', $js_autodetect_results);
+if(!sqGetGlobalVar('squirrelmail_language', $squirrelmail_language) || $squirrelmail_language == '') {
+	$squirrelmail_language = $squirrelmail_default_language;
 }
+
 /* end of get globals */
 
 set_up_language($squirrelmail_language, true);