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

WordPress Trac noreply at wordpress.org
Tue Apr 6 03:42:16 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 fixed-major needs-testing  |     Focuses:
-------------------------------------------------+------------------------
Changes (by peterwilsoncc):

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


Comment:

 In [changeset:"50662" 50662]:
 {{{
 #!CommitTicketReference repository="" revision="50662"
 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.
 Merges [50586] to the 5.7 branch.
 Fixes #52826.
 }}}

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


More information about the wp-trac mailing list