Bug #14364 Updated: Unable to clear out an individual variable in a session

ID: 14364 Updated by: mfischer Reported By: info@somnioworld.com Old Status: Closed Status: Feedback Bug Type: Session related Operating System: FreeBSD 4.2 PHP Version: 4.1.0 New Comment: Feedback. Please provide a short, reproduceable copy&paste sample. Previous Comments: ------------------------------------------------------------------------ [2001-12-07 13:25:29] info@somnioworld.com 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] derick@php.net 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] info@somnioworld.com 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? ------------------------------------------------------------------------ [2001-12-07 01:28:45] derick@php.net Are you sure you did that after the fix? file /ext/session/php_session.h should include this (around line 168): #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); \ } Does it do that, and did you rebuild from a fresh tree? Derick ------------------------------------------------------------------------ [2001-12-06 22:08:10] info@somnioworld.com We downloaded and installed the CVS version today (version 4.2) and we are still having the same issue. ------------------------------------------------------------------------ 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