[wp-trac] [WordPress Trac] #48975: Fix unhandled upper/lower case change in wp_unique_filename()

WordPress Trac noreply at wordpress.org
Sun Dec 15 18:30:06 UTC 2019


#48975: Fix unhandled upper/lower case change in wp_unique_filename()
-------------------------------------------------+-------------------------
 Reporter:  azaozz                               |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  high                                 |   Milestone:  5.3.2
Component:  Upload                               |     Version:  5.3.1
 Severity:  major                                |  Resolution:
 Keywords:  has-patch 2nd-opinion needs-testing  |     Focuses:
  has-unit-tests                                 |
-------------------------------------------------+-------------------------
Description changed by azaozz:

Old description:

> Introduced in #42437, [46822].
>
> On non case-sensitive file systems it may fail to match a file when the
> uploaded wile is with upper case extension. Also if a file with upper
> case extension is uploaded, and the file name matches exactly an existing
> file name that has dimensions-like ending, it may fail to rename the file
> causing a fatal error.
>
> Both cases seem very rare but worth fixing asap as uploading a file with
> upper case (or mixed case) extension may result in a fatal error.

New description:

 Introduced in #42437, [46822].

 On non case-sensitive file systems it may fail to match a file name when
 the uploaded file is with upper case extension. Also if a file with upper
 case extension is uploaded, and the file name matches exactly an existing
 file name that has dimensions-like ending, it may fail to rename the file
 causing a fatal error.

 Both cases seem very rare but worth fixing asap as uploading a file with
 upper case (or mixed case) extension may result in a fatal error.

--

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/48975#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list