Make WordPress Core

Ticket #54385: 54385.4.diff

File 54385.4.diff, 2.6 KB (added by adamsilverstein, 3 years ago)
  • src/wp-admin/includes/image.php

    diff --git src/wp-admin/includes/image.php src/wp-admin/includes/image.php
    index d8d5d5b34b..29bab6dfed 100644
    function wp_generate_attachment_metadata( $attachment_id, $file ) { 
    647647 * @since 2.5.0
    648648 *
    649649 * @param string $str
    650  * @return int|float
     650 * @return int|float Returns 0 on failure.
    651651 */
    652652function wp_exif_frac2dec( $str ) {
    653         if ( false === strpos( $str, '/' ) ) {
    654                 return $str;
     653        // Fractions must contain a single `/`.
     654        if ( false === strpos( $str, '/' ) || substr_count( $str, '/' ) > 1 ) {
     655                return 0;
    655656        }
    656657
    657658        list( $numerator, $denominator ) = explode( '/', $str );
    658         if ( ! empty( $denominator ) ) {
     659
     660        // Ensure both the numerator and the denominator are numbers.
     661        if ( ! is_numeric( $numerator ) || ! is_numeric( $denominator ) ) {
     662                return 0;
     663        }
     664
     665        // The denominator must not be 0.
     666        if ( $denominator != 0 ) {
    659667                return $numerator / $denominator;
    660668        }
    661         return $str;
     669        return 0;
    662670}
    663671
    664672/**
  • tests/phpunit/tests/image/functions.php

    diff --git tests/phpunit/tests/image/functions.php tests/phpunit/tests/image/functions.php
    index 488731a0e8..deddf164c2 100644
    class Tests_Image_Functions extends WP_UnitTestCase { 
    700700                        unlink( $temp_dir . $size['file'] );
    701701                }
    702702        }
     703
     704        /**
     705         * Test for wp_exif_frac2dec verified that it properly handles edge cases
     706         * and always returns an int or float, 0 for failures.
     707         *
     708         * @ticket 54385
     709         * @dataProvider data_test_wp_exif_frac2dec
     710         */
     711        public function test_wp_exif_frac2dec( $fraction, $expect ) {
     712                $this->assertSame( $expect, wp_exif_frac2dec( $fraction ) );
     713        }
     714
     715
     716        /**
     717         * Data provider for testing `wp_exif_frac2dec()`.
     718         *
     719         * @return array {
     720         *     Arguments for testing `wp_exif_frac2dec()`.
     721         *
     722         *     @type string|null        $fraction       The fraction string to run.
     723         *     @type int|float       $expected    The resulting expected value.
     724         *
     725         */
     726        public function data_test_wp_exif_frac2dec() {
     727                global $wpdb;
     728
     729                return array(
     730                        array(
     731                                '0/0',
     732                                0,
     733                        ),
     734                        array(
     735                                '0/abc',
     736                                0,
     737                        ),
     738                        array(
     739                                '0.0',
     740                                0,
     741                        ),
     742                        array(
     743                                '010',
     744                                0,
     745                        ),
     746                        array(
     747                                10.123,
     748                                0,
     749                        ),
     750                        array(
     751                                '50/100',
     752                                0.5,
     753                        ),
     754                        array(
     755                                '25/100',
     756                                .25,
     757                        ),
     758                        array(
     759                                '0',
     760                                0,
     761                        ),
     762                        array(
     763                                '100/0',
     764                                0,
     765                        ),
     766                        array(
     767                                'path/to/file',
     768                                0,
     769                        ),
     770                        array(
     771                                '123notafraction',
     772                                0,
     773                        ),
     774                        array(
     775                                '/',
     776                                0,
     777                        ),
     778                        array(
     779                                '1/2/3',
     780                                0,
     781                        ),
     782                        array(
     783                                '///',
     784                                0,
     785                        ),
     786                );
     787        }
     788
    703789}