The current version of apcCache::delete loops on a possibly empty array.

This small patch is similar to the one applied for 1.1.2.13 in the D6 branch for this specific problem. It does not tackle the other apc issues, though.

CommentFileSizeAuthor
#1 cacherouter.patch1 KBfgm

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
StatusFileSize
new cacherouter.patch1 KB

Grrr, missed patch attachment.

alippai’s picture

alippai commented
Status: Needs review » Reviewed & tested by the community

I don't know, why apc_fetch return empty string - if we can drop theese strings this is RTBC.

EDIT, checking for &$success or for return false is not enough :/

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented

seeing this issue as well whenever we clear Drupal's cache, at least the patch will test for an array before foreach()ing.

warning: Invalid argument supplied for foreach() in sites/all/modules/cacherouter/engines/apc.php on line 121.
slantview’s picture

slantview commented
Status: Reviewed & tested by the community » Fixed

This has already been fixed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.