#26527 [NEW]: Either GC code is wrong or GC documentation is wrong

From: jneil at myersinternet dot com Operating system: Linux PHP version: 4.3.3 PHP Bug Type: Session related Bug description: Either GC code is wrong or GC documentation is wrong Description: ------------ Both the code documentation in mod_files.c (on line 207) and the PHP manual indicate that the garbage cleanup should take place based on a given session file's last access time. However, the element from "struct stat" being used in mod_files.c on line 209 is sbuf.st_mtime (the last modification time) instead of sbuf.st_atime. Either the code is incorrect or the documentation is incorrect. If we can get this synced up properly, then those of us trying to debug session timeout issues will have a much easier time deciding which parameters are governing session timeouts. The coding change may have been to fix problems on a particular platform but the result has been to make the documentation misleading. -- Edit bug report at http://bugs.php.net/?id=26527&edit=1 -- Try a CVS snapshot (php4): http://bugs.php.net/fix.php?id=26527&r=trysnapshot4 Try a CVS snapshot (php5): http://bugs.php.net/fix.php?id=26527&r=trysnapshot5 Fixed in CVS: http://bugs.php.net/fix.php?id=26527&r=fixedcvs Fixed in release: http://bugs.php.net/fix.php?id=26527&r=alreadyfixed Need backtrace: http://bugs.php.net/fix.php?id=26527&r=needtrace Need Reproduce Script: http://bugs.php.net/fix.php?id=26527&r=needscript Try newer version: http://bugs.php.net/fix.php?id=26527&r=oldversion Not developer issue: http://bugs.php.net/fix.php?id=26527&r=support Expected behavior: http://bugs.php.net/fix.php?id=26527&r=notwrong Not enough info: http://bugs.php.net/fix.php?id=26527&r=notenoughinfo Submitted twice: http://bugs.php.net/fix.php?id=26527&r=submittedtwice register_globals: http://bugs.php.net/fix.php?id=26527&r=globals PHP 3 support discontinued: http://bugs.php.net/fix.php?id=26527&r=php3 Daylight Savings: http://bugs.php.net/fix.php?id=26527&r=dst IIS Stability: http://bugs.php.net/fix.php?id=26527&r=isapi Install GNU Sed: http://bugs.php.net/fix.php?id=26527&r=gnused Floating point limitations: http://bugs.php.net/fix.php?id=26527&r=float