Make WordPress Core

Ticket #55966: 55966.2.diff

File 55966.2.diff, 3.8 KB (added by johnregan3, 2 years ago)

Minor update.

  • src/wp-includes/kses.php

    diff --git src/wp-includes/kses.php src/wp-includes/kses.php
    index b32df1beaa..1c581b3714 100644
    function safecss_filter_attr( $css, $deprecated = '' ) { 
    24672467                }
    24682468
    24692469                if ( $found ) {
    2470                         // Allow CSS calc().
    2471                         $css_test_string = preg_replace( '/calc\(((?:\([^()]*\)?|[^()])*)\)/', '', $css_test_string );
     2470                        // Allow some CSS functions.
     2471                        $css_test_string = preg_replace( '/\b(?:calc|min|max|minmax|clamp)\(((?:\([^()]*\)?|[^()])*)\)/', '', $css_test_string );
     2472
    24722473                        // Allow CSS var().
    2473                         $css_test_string = preg_replace( '/\(?var\(--[a-zA-Z0-9_-]*\)/', '', $css_test_string );
     2474                        $css_test_string = preg_replace( '/\(?var\(--[\w\-\()[\]\,\s]*\)/', '', $css_test_string );
    24742475
    24752476                        // Check for any CSS containing \ ( & } = or comments,
    24762477                        // except for url(), calc(), or var() usage checked above.
  • tests/phpunit/tests/kses.php

    diff --git tests/phpunit/tests/kses.php tests/phpunit/tests/kses.php
    index e6cac09c3b..4683eca1be 100644
    EOF; 
    935935         * @ticket 37248
    936936         * @ticket 42729
    937937         * @ticket 48376
     938         * @ticket 55966
    938939         * @dataProvider data_test_safecss_filter_attr
    939940         *
    940941         * @param string $css      A string of CSS rules.
    EOF; 
    11201121                                'css'      => 'color: rgb( 100, 100, 100, .4 )',
    11211122                                'expected' => '',
    11221123                        ),
     1124                        // Allow min().
     1125                        array(
     1126                                'width: min(50%, 400px)',
     1127                                'width: min(50%, 400px)',
     1128                        ),
     1129                        // Allow max().
     1130                        array(
     1131                                'width: max(50%, 40rem)',
     1132                                'width: max(50%, 40rem)',
     1133                        ),
     1134                        // Allow minmax().
     1135                        array(
     1136                                'width: minmax(100px, 50%)',
     1137                                'width: minmax(100px, 50%)',
     1138                        ),
     1139                        // Allow clamp().
     1140                        array(
     1141                                'width: clamp(100px, 50%, 100vw)',
     1142                                'width: clamp(100px, 50%, 100vw)',
     1143                        ),
     1144                        // Combined CSS function names.
     1145                        array(
     1146                                'width: calcmax(100px + 50%)',
     1147                                '',
     1148                        ),
     1149                        // Allow calc().
     1150                        array(
     1151                                'width: calc(2em + 3px)',
     1152                                'width: calc(2em + 3px)',
     1153                        ),
     1154                        // Allow var().
     1155                        array(
     1156                                'padding: var(--wp-var1) var(--wp-var2)',
     1157                                'padding: var(--wp-var1) var(--wp-var2)',
     1158                        ),
     1159                        // Allow var() with fallback (commas).
     1160                        array(
     1161                                'padding: var(--wp-var1, 10px)',
     1162                                'padding: var(--wp-var1, 10px)',
     1163                        ),
     1164                        // Allow var() with fallback var().
     1165                        array(
     1166                                'background-color: var(--wp-var, var(--wp-var-fallback, pink))',
     1167                                'background-color: var(--wp-var, var(--wp-var-fallback, pink))',
     1168                        ),
     1169                        // Allow calc() with var().
     1170                        array(
     1171                                'margin-top: calc(var(--wp-var1) * 3 + 2em)',
     1172                                'margin-top: calc(var(--wp-var1) * 3 + 2em)',
     1173                        ),
     1174                        // Malformed min, no closing `)`.
     1175                        array(
     1176                                'width: min(3em + 10px',
     1177                                '',
     1178                        ),
     1179                        // Malformed max, no closing `)`.
     1180                        array(
     1181                                'width: max(3em + 10px',
     1182                                '',
     1183                        ),
     1184                        // Malformed minmax, no closing `)`.
     1185                        array(
     1186                                'width: minmax(3em + 10px',
     1187                                '',
     1188                        ),
     1189                        // Malformed calc, no closing `)`.
     1190                        array(
     1191                                'width: calc(3em + 10px',
     1192                                '',
     1193                        ),
     1194                        // Malformed var, no closing `)`.
     1195                        array(
     1196                                'width: var(--wp-var1',
     1197                                '',
     1198                        ),
    11231199                );
    11241200        }
    11251201
    EOF; 
    13011377                                'background: red',
    13021378                        ),
    13031379
    1304                         // CSS calc().
    1305                         array(
    1306                                 'width: calc(2em + 3px)',
    1307                                 'width: calc(2em + 3px)',
    1308                         ),
    1309 
    1310                         // CSS variable.
    1311                         array(
    1312                                 'padding: var(--wp-var1) var(--wp-var2)',
    1313                                 'padding: var(--wp-var1) var(--wp-var2)',
    1314                         ),
    1315 
    1316                         // CSS calc() with var().
    1317                         array(
    1318                                 'margin-top: calc(var(--wp-var1) * 3 + 2em)',
    1319                                 'margin-top: calc(var(--wp-var1) * 3 + 2em)',
    1320                         ),
    1321 
    13221380                        /*
    13231381                         * Invalid use cases.
    13241382                         */
    EOF; 
    13821440                                'background-image: url( "http://example.com );',
    13831441                                '',
    13841442                        ),
    1385 
    1386                         // Malformed calc, no closing `)`.
    1387                         array(
    1388                                 'width: calc(3em + 10px',
    1389                                 '',
    1390                         ),
    1391 
    1392                         // Malformed var, no closing `)`.
    1393                         array(
    1394                                 'width: var(--wp-var1',
    1395                                 '',
    1396                         ),
    13971443                );
    13981444        }
    13991445