Project:Cache Router
Version:7.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

Comments

#1

Grrr, missed patch attachment.

AttachmentSize
cacherouter.patch 1 KB

#2

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 :/

#3

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.

#4

Status:reviewed & tested by the community» fixed

This has already been fixed.

#5

Status:fixed» closed (fixed)

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

nobody click here