Active
Project:
Plupload integration
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
30 Mar 2011 at 18:51 UTC
Updated:
30 Mar 2011 at 18:51 UTC
This was noticed by @effulgentsia.
Currently when a temporary file is actually uploaded to the server by Plupload, we validate it with preg_match('/^\w+\.tmp$/', $file_name). When that temporary file is going through the form API validation process before being copied, though, we validate it with a looser set of criteria.
Both should be safe, but for consistency we should use the same (more stringent) file name validation in both places.