Re: cvs: php4 /ext/session php_session.h
| From: | Thies C. Arntzen | Date: | Thu, 06 Dec 2001 14:31:44 +0000 |
| Subject: | Re: cvs: php4 /ext/session php_session.h | ||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-8464@lists.php.net to get a copy of this message | ||
On Thu, Dec 06, 2001 at 02:20:20PM -0000, Zeev Suraski wrote:
> zeev Thu Dec 6 09:20:20 2001 EDT
>
> Modified files:
> /php4/ext/session php_session.h
> Log:
> Fix session_unregister()
we need more regression tests...
do you have a code snippet that showed the wrong behaviour?
if yes - i'll write a testcase!
tc
>
>
> Index: php4/ext/session/php_session.h
> diff -u php4/ext/session/php_session.h:1.64 php4/ext/session/php_session.h:1.65
> --- php4/ext/session/php_session.h:1.64 Thu Dec 6 08:33:10 2001
> +++ php4/ext/session/php_session.h Thu Dec 6 09:20:20 2001
> @@ -166,8 +166,12 @@
>
> #define PS_ADD_VAR(name) PS_ADD_VARL(name, strlen(name))
>
> -#define PS_DEL_VARL(name,namelen) \
> - zend_hash_del(&PS(vars), name, namelen + 1);
> +#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); \
> + }
> +
>
> #define PS_ENCODE_VARS \
> char *key; \
>
>
>
> --
> 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
>