# [PATCH] fix #23432

- Thread: https://externals.io/message/7668
- Original archive: https://news-web.php.net/php.internals/7668
- Started by Nuno Lopes on 2004-02-09T16:09:02+00:00
- Messages: 4
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #7668 — Nuno Lopes — 2004-02-09T16:09:02+00:00

URL: https://externals.io/message/7668#7668

Hi,

As I don't have sufficient karma, here is a patch to fix bug #23432 (PHP
using mtime instead of atime to clean session files).


Nuno

---

## #7671 — Timm Friebe — 2004-02-09T16:54:11+00:00

URL: https://externals.io/message/7668#7671 · In reply to: #7668

> Hi,
> 
> As I don't have sufficient karma, here is a patch to fix bug #23432 (PHP
> using mtime instead of atime to clean session files).

Might be more correct, but what about filesystems mounted with
"noatime"?

- Timm

---

## #7675 — Jani Taskinen — 2004-02-09T17:48:01+00:00

URL: https://externals.io/message/7668#7675 · In reply to: #7668

>Hi,
>
>As I don't have sufficient karma, here is a patch to fix bug #23432 (PHP
>using mtime instead of atime to clean session files).

----------------------------
revision 1.76
date: 2002/08/17 20:32:26;  author: zeev;  state: Exp;  lines: +2 -2
Use mtime instead of atime, as we always update the session file anyway.
----------------------------

     --Jani

---

## #7677 — Nuno Lopes — 2004-02-09T18:30:56+00:00

URL: https://externals.io/message/7668#7677 · In reply to: #7675

> ----------------------------
> revision 1.76
> date: 2002/08/17 20:32:26;  author: zeev;  state: Exp;  lines: +2 -2
> Use mtime instead of atime, as we always update the session file anyway.
> ----------------------------
>
>      --Jani


Sorry... I hadn't see that. After submitting the patch I've found that
message and then I've updated the docs.


Nuno