Re: cvs: php4(PHP_4_0_7) /ext/standard var.c

On Sat, Dec 01, 2001 at 11:51:26PM +0200, Zeev Suraski wrote: > By the way, if you (personally:) give these patches a second thorough look, > perhaps they can go into RC5 (it does make working with $_SESSION slightly > better). I'm simply afraid that because I'm not too familiar with the > session code, I may have introduced some quirk or bug... i'm not familiar with that code as well. but as this is a problem with the now recommented register_globals=off setting i think it should be fixed in 4.1. qa-team - question is: merge those fixes or not? tc > > Zeev > > At 20:33 01/12/2001, Thies C. Arntzen wrote: > >On Sat, Dec 01, 2001 at 05:11:18PM +0100, Sebastian Bergmann wrote: > >> Sebastian Bergmann wrote: > >> > "Thies C. Arntzen" wrote: > >> >> bug id# ? > >> > > >> > Not available, since I told this Sascha and Zeev directly on IRC and > >> > by mail. > >> > >> Here the mail I sent to them. Zeev fixed this in HEAD with the two > >> most recent patches to > >> > >> http://cvs.php.net/cvs.php/php4/ext/session/session.c > > > > zeev, should this be added to 4.1? > > > >> > >> Environment: PHP 4.2.0-dev (current HEAD from CVS) as CGI, Win32. > >> > >> This works > >> > >> <?php > >> ini_set('register_globals', 0); > >> session_register('foo'); > >> echo ++$_SESSION['foo']; > >> ?> > >> > >> while this does not > >> > >> <?php > >> ini_set('register_globals', 1); > >> session_register('foo'); > >> echo ++$_SESSION['foo']; > >> ?> > >> > >> Why is $_SESSION not populated when register_globals is on? > >> > >> -- > >> Sebastian Bergmann > >> http://sebastian-bergmann.de/ > >> http://phpOpenTracker.de/ > >> > >> Did I help you? Consider a gift: > >> http://wishlist.sebastian-bergmann.de/ > >> > >> -- > >> PHP CVS Mailing List (http://www.php.net/) > >> To unsubscribe, e-mail: php-cvs-unsubscribe@lists.php.net > >> For additional commands, e-mail: php-cvs-help@lists.php.net > >> To contact the list administrators, e-mail: php-list-admin@lists.php.net > >> >