ID: 14364 Updated by: sniper Reported By: [EMAIL PROTECTED] Old Status: Open Status: Closed Bug Type: Session related Operating System: FreeBSD 4.2 PHP Version: 4.1.0 New Comment:
This is fixed in latest CVS. I can't reproduce it whatever the register_globals is. --Jani Previous Comments: ------------------------------------------------------------------------ [2001-12-07 14:13:41] [EMAIL PROTECTED] Ok, it looks like I have everything resolved. In the previous version of PHP (4.0.5) I was able to clear out a session variable by just using the unset() function. Now with the new CVS it appears that I have to session_unregister() and unset() the session variable to clear it out. Thank you for your support! ------------------------------------------------------------------------ [2001-12-07 13:34:17] [EMAIL PROTECTED] Feedback. Please provide a short, reproduceable copy&paste sample. ------------------------------------------------------------------------ [2001-12-07 13:25:29] [EMAIL PROTECTED] I did check again and you are right, the lines of code you posted are in the version I installed. However, I am still having the issues with the session. I have tested it several times and when I try unsetting session variables and unregistering session variables it still doesn't work. Let me know what I need to do to help get this resolved. I truely appreciate the help ------------------------------------------------------------------------ [2001-12-07 12:46:39] [EMAIL PROTECTED] It is certainly in the php 4.2.0 cvs tree, as I just downloaded it here. with the instructions on that site, you should have the correct version too (with the fix). Can you please check again of lines 169-173 in php4/ext/session/php_session.h have these lines of codE: #define PS_DEL_VARL(name,namelen) \ zend_hash_del(&PS(vars), name, namelen+1); \ if (PS(http_session_vars)) { \ zend_hash_del(Z_ARRVAL_P(PS(http_session_vars)), name, namelen+1); \ } regards, Derick ------------------------------------------------------------------------ [2001-12-07 11:45:32] [EMAIL PROTECTED] Those lines are not in my /ext/session/php_session.h I installed the CVS version this afternoon. I followed the instruction on php.net's site: http://www.php.net/anoncvs.php I did it from a new dir, so the tree is fresh. I was also running php4.1 rc5 prior to the upgrade and now after the upgrade I am running php4.2 which is the cvs version I got today. Where should I get the CVS version that has the fix? Or can I just add the lines in that you posted to the source to get it to work correctly? What do you recommend? ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/?id=14364 Edit this bug report at http://bugs.php.net/?id=14364&edit=1 -- PHP Development Mailing List <http://www.php.net/> To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] To contact the list administrators, e-mail: [EMAIL PROTECTED]