[20925] Fix: Stop verify if there is no session to verify

Sigurd Nes <[email protected]>
Newsgroups gmane.comp.web.phpgroupware.cvs
Message-ID <[email protected]>
Revision: 20925
          http://svn.sv.gnu.org/viewvc/?view=rev&root=phpgroupware&revision=20925
Author:   sigurdne
Date:     2010-01-05 21:20:28 +0000 (Tue, 05 Jan 2010)
Log Message:
-----------
Fix: Stop verify if there is no session to verify

Modified Paths:
--------------
    people/sigurdne/modules/phpgwapi/trunk/inc/class.sessions.inc.php

Modified: people/sigurdne/modules/phpgwapi/trunk/inc/class.sessions.inc.php
===================================================================
--- people/sigurdne/modules/phpgwapi/trunk/inc/class.sessions.inc.php	2010-01-05 21:19:03 UTC (rev 20924)
+++ people/sigurdne/modules/phpgwapi/trunk/inc/class.sessions.inc.php	2010-01-05 21:20:28 UTC (rev 20925)
@@ -271,6 +271,7 @@
 			}
 			$GLOBALS['phpgw_info']['user']['account_id'] = $this->_account_id;
 			$accounts->set_account($this->_account_id);
+
 			session_start();
 			$this->_sessionid = session_id();
 
@@ -936,7 +937,8 @@
 
 			if ( !strlen(session_id()) )
 			{
-				session_start();
+				throw new Exception("sessions::register_session() - No value for session_id()");
+//				session_start();
 			}
 
 			$_SESSION['phpgw_session'] = array
@@ -1100,6 +1102,11 @@
 				$sessionid = phpgw::get_var(session_name());
 			}
 
+			if(!$sessionid)
+			{
+				return false;
+			}
+
 			$this->_sessionid = $sessionid;
 
 			$session = $this->read_session($sessionid);
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.