[wp-trac] [WordPress Trac] #6450: wp_handle_upload does not appear to handle file move errors correctly

WordPress Trac wp-trac at lists.automattic.com
Sat Mar 29 18:30:23 GMT 2008


#6450: wp_handle_upload does not appear to handle file move errors correctly
---------------------------+------------------------------------------------
 Reporter:  simonwheatley  |       Owner:  anonymous               
     Type:  defect         |      Status:  new                     
 Priority:  normal         |   Milestone:  2.5.1                   
Component:  General        |     Version:  2.5                     
 Severity:  normal         |    Keywords:  has_patch, needs_testing
---------------------------+------------------------------------------------
 ''wp_handle_upload'' uses a configurable error handler for all errors,
 except in the case where it fails to move the file to the uploads
 directory. I think this is an ommission, which the attached patch fixes.
 The patch also corrects an instance of ''printf'' to ''sprintf''.

 To reproduce:

 1. Set your upload directory & subdirectories to readonly (chmod -R 444
 /your/uploads/)

 2. Attempt to upload a file through the 2.5 SWF uploader

 3. Observe the generic error message "An error occurred in the upload.
 Please try again later."

-- 
Ticket URL: <http://trac.wordpress.org/ticket/6450>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list