[wp-trac] [WordPress Trac] #52826: New wp_getimagesize() causing unexpected failures

WordPress Trac noreply at wordpress.org
Fri Mar 26 00:07:40 UTC 2021


#52826: New wp_getimagesize() causing unexpected failures
--------------------------------------+------------------------
 Reporter:  terriann                  |       Owner:  whyisjake
     Type:  defect (bug)              |      Status:  closed
 Priority:  normal                    |   Milestone:  5.7.1
Component:  Media                     |     Version:  5.7
 Severity:  critical                  |  Resolution:  fixed
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+------------------------
Changes (by peterwilsoncc):

 * status:  reopened => closed
 * resolution:   => fixed


Comment:

 In [changeset:"50586" 50586]:
 {{{
 #!CommitTicketReference repository="" revision="50586"
 Media: Conditionally pass 2nd parameter to `getimagesize()`.

 In the wrapper function `wp_getimagesize()` check if the second parameter
 was passed before sending it to the PHP function `getimagesize()`.

 The PHP function has a different execution path depending on the number of
 parameters passed, this ensures the wrapper function follows the
 appropriate path.

 Follow up to [50552].
 Props azaozz, hellofromtonya, Mista-Flo, peterwilsoncc, rinatkhaziev,
 RogerTheriault, SergeyBiryukov, terriann, whyisjake.
 Fixes #52826.
 }}}

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


More information about the wp-trac mailing list