php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #9564 wrong increment nrdels in PS_GC_FUNC(mm)
Submitted: 2001-03-05 11:25 UTC Modified: 2001-08-30 11:49 UTC
From: eiji at papanui dot ddt dot or dot jp Assigned:
Status: Closed Package: Session related
PHP Version: 4.0 Latest CVS (05/03/2001) OS: Any
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem:
24 + 3 = ?
Subscribe to this entry?

 
 [2001-03-05 11:25 UTC] eiji at papanui dot ddt dot or dot jp
It's wrong increment nrdels in PS_GC_FUNC(mm)

'*nrdels++' incremnts nrdels as pointer,
but nrdels is allocated on the stack in php_session_start(PSLS_D).

This is a patch for a local copy of my repositry,

--- mod_mm.c    2001/01/13 10:54:33     1.1.1.2
+++ mod_mm.c    2001/03/05 16:11:46
@@ -333,7 +333,7 @@
                        ps_mm_debug("looking at %s\n", sd->key);
                        if ((now - sd->ctime) > maxlifetime) {
                                ps_sd_destroy(data, sd);
-                               *nrdels++;
+                               (*nrdels)++;
                        }
                }

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2001-03-16 17:39 UTC] sas@php.net
Thanks for your report. The specific code was changed in the past and was not affected by that bug anymore.
 [2001-08-30 11:49 UTC] eiji at papanui dot ddt dot or dot jp
Thank you for fixed this bug at revision 1.21 (19th July 2001).

 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Fri Mar 29 11:01:29 2024 UTC