Closed (fixed)
Project:
Security Review
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Feb 2012 at 20:03 UTC
Updated:
13 Dec 2012 at 23:30 UTC
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
Comment #1
gregglesThanks for your bug report and for helping make this a better module. Could you provide this as a patch?
Comment #2
coltraneThis 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.
Comment #3
brianmercer commentedThe current 6.x-1.x-dev solves this issue, thanks.