Can you provide the shortest sample script, that reproduce that bug?

Fixed in CVS. BTW, 4.1.0 does not exist yet, so how can you report a bug for it?
This fix will be in the 4.1.0 release.

Derick

Sample script:

// script to create session variable
session_start();
session_register("objectA");
$objectA = new classA();

// script to unset session variable
session_start();
session_unregister("objectA");
unset($objectA);

// script to test for session variable
session_start();
if(isset($objectA))
{
  echo "TEST PASSED";
}


Doing a session_destroy() then unset() of all the session variables seems to work, but in the above code I wish to just unset() a single variable in the session but keep the session and the remaining variables alive.

No worries! It's already fixed in CVS.

We downloaded and installed the CVS version today (version 4.2) and we are still having the same issue.

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

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?

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

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

Feedback.

Please provide a short, reproduceable copy&paste sample.

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!

This is fixed in latest CVS. I can't reproduce it whatever the 
register_globals is.

--Jani