My file system path is "files".

The function security_review_check_file_perms() fails because this code:

$file_path = substr($file_path, strrpos($file_path, '/') + 1);

creates an ignore pattern of "iles" due to the lack of a slash in my path.

Something like this would work:

$file_path = substr($file_path, (strpos($file_path, '/') ? strrpos($file_path, '/') + 1 : 0) );

Comments

greggles’s picture

Thanks for your bug report and for helping make this a better module. Could you provide this as a patch?

coltrane’s picture

Version: 6.x-1.2 » 6.x-1.x-dev
Status: Active » Fixed

This should be fixed with the commits from http://drupal.org/node/1645752

Please try out 1.x-dev releases to verify and expect a 1.3 release soon.

brianmercer’s picture

The current 6.x-1.x-dev solves this issue, thanks.

Status: Fixed » Closed (fixed)

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