[SM-CVS] CVS: squirrelmail/include validate.php,1.1,1.2

[Erin S. <siz...@us...>]

Update of /cvsroot/squirrelmail/squirrelmail/include
In directory usw-pr-cvs1:/tmp/cvs-serv398/squirrelmail/include

Modified Files:
	validate.php 
Log Message:
Update for session.auto_start:
If session is automatically started before the class
definitions are loaded, many functions (compose, certain
parts of read_body, download, etc.) will fail with the
following error:
The script tried to execute a method or access a
property of an incomplete object.

Still to do: ensure that we close the session using 
session_write_close() as soon as the page is finished
loading session-specific variables to reduce
contention between frames.



Index: validate.php
===================================================================
RCS file: /cvsroot/squirrelmail/squirrelmail/include/validate.php,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -w -r1.1 -r1.2
--- validate.php	6 Sep 2002 01:16:35 -0000	1.1
+++ validate.php	23 Sep 2002 15:15:22 -0000	1.2
@@ -11,7 +11,22 @@
 
 /* include the mime class before the session start ! otherwise we can't store
  * messages with a session_register.
+ *
+ * From http://www.php.net/manual/en/language.oop.serialization.php:
+ *   In case this isn't clear:
+ *   In 4.2 and below: 
+ *      session.auto_start and session objects are mutually exclusive.
+ *
+ * We need to load the classes before the session is started, 
+ * except that the session could be started automatically 
+ * via session.auto_start. So, we'll close the session, 
+ * then load the classes, and reopen the session which should 
+ * make everything happy.  
+ *
+ * ** Note this means that for the 1.3.2 release, we should probably
+ * recommend that people set session.auto_start=0 to avoid this altogether.
  */
+session_write_close();
  
 /* SquirrelMail required files. */
 require_once(SM_PATH . 'class/mime.class.php');