Ticket #4141 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

default cache behavior leaves tmp files when cache dir unwritable

Reported by: andy Assigned to: rob1n
Priority: high Milestone: 2.2
Component: General Version: 2.1
Severity: normal Keywords: has-patch commit
Cc:

Description

As reported by Sabin Iacob [wp-hackers] Wordpress object cache, the files in /tmp/ are not removed.

It seems to happen in cache.php, where there is no unlink in the second clause of the if statement of line 343. The tmp files are unusable so they should be removed regardless of success.

Attachments

cache.diff (0.5 kB) - added by andy on 04/12/07 09:31:04.

Change History

04/12/07 09:24:32 changed by Nazgul

The thread: http://comox.textdrive.com/pipermail/wp-hackers/2007-April/011845.html

04/12/07 09:31:04 changed by andy

  • attachment cache.diff added.

04/12/07 15:56:36 changed by rob1n

  • keywords set to has-patch commit.
  • owner changed from anonymous to rob1n.

+1. Looks good.

04/12/07 16:31:50 changed by rob1n

  • status changed from new to closed.
  • resolution set to fixed.

(In [5255]) Delete the cache in tmp even if the copy failed. Props andy. fixes #4141