Browse Source

Replace all session_start() calls with sqsession_is_active(). The latter
is a conditional session_start which will only start a session if none
has been before. This will make us compatible with the upcoming PHP 4.3.3
release. Thanks Phil Driscoll for the tip.

Thijs Kinkhorst 22 years ago
parent
commit
39f096f36e
4 changed files with 6 additions and 4 deletions
  1. 3 1
      ChangeLog
  2. 1 1
      include/validate.php
  3. 1 1
      src/redirect.php
  4. 1 1
      src/webmail.php

+ 3 - 1
ChangeLog

@@ -76,7 +76,9 @@ Version 1.5.0 -- CVS
   - sqimap_msgs_list_copy() no longer deletes messages copied.
   - Workaround for Mozilla bug #200412 in order to show multipart/related html mail.
   - Fix for disapearing '0' from decoded strings (bug #784193)
-  
+  - Replace all session_start() calls with sqsession_is_active() to be compatible
+    with upcoming PHP 4.3.3.
+
 
 **************************************
 *** SquirrelMail Stable Series 1.4 ***

+ 1 - 1
include/validate.php

@@ -50,7 +50,7 @@ if(isset($session_name) && $session_name) {
     ini_set('session.name' , 'SQMSESSID');
 }
 
-session_start();
+sqsession_is_active();
 
 require_once(SM_PATH . 'functions/i18n.php');
 require_once(SM_PATH . 'functions/auth.php');

+ 1 - 1
src/redirect.php

@@ -37,7 +37,7 @@ header('Pragma: no-cache');
 $location = get_location();
 
 session_set_cookie_params (0, $base_uri);
-session_start();
+sqsession_is_active();
 
 sqsession_unregister ('user_is_logged_in');
 sqsession_register ($base_uri, 'base_uri');

+ 1 - 1
src/webmail.php

@@ -31,7 +31,7 @@ if (!function_exists('sqm_baseuri')){
 }
 $base_uri = sqm_baseuri();
 
-session_start();
+sqsession_is_active();
 
 sqgetGlobalVar('username', $username, SQ_SESSION);
 sqgetGlobalVar('delimiter', $delimiter, SQ_SESSION);