Make WordPress Core

Ticket #21523: 21523.3.patch

File 21523.3.patch, 5.2 KB (added by chriscct7, 9 years ago)

Improves upon second patch by adding more strict checking on API return

  • wp-admin/credits.php

     
    3737
    3838                $results = json_decode( wp_remote_retrieve_body( $response ), true );
    3939
    40                 if ( ! is_array( $results ) )
     40                if ( ! is_array( $results ) ){
    4141                        return false;
     42                }
    4243
     44                if ( empty( $results['groups'] ) || !is_array( $results['groups'] ) ){
     45                        return false;
     46                } else {
     47                        foreach ( $results['groups'] as $group_slug => $group_data ) {
     48                                if ( empty( $group_data['name'] )
     49                                         && 'contributing-developers' !== $group_slug
     50                                         && 'recent-rockstars' !== $group_slug ){
     51                                        return false;
     52                                } elseif ( empty( $group_data['type'] ) ){
     53                                        return false;
     54                                } elseif ( empty( $group_data['data'] ) ){
     55                                        return false;
     56                                }
     57                        }
     58                }
     59
     60                if ( empty( $results['data'] ) || !is_array( $results['data'] ) ){
     61                        return false;
     62                } elseif (              empty( $results['data']['profiles'] )
     63                                        ||  empty( $results['data']['version'] )
     64                                        || !is_string( $results['data']['profiles'] )
     65                                        || !is_string( $results['data']['version'] ) ) {
     66                        return false;
     67                }
     68
    4369                set_site_transient( 'wordpress_credits_' . $locale, $results, DAY_IN_SECONDS );
    4470        }
    4571
     
    7197 * @return string Link to the external library.
    7298 */
    7399function _wp_credits_build_object_link( &$data ) {
    74         $data = '<a href="' . esc_url( $data[1] ) . '">' . $data[0] . '</a>';
     100        $data = '<a href="' . esc_url( $data[1] ) . '">' . esc_html( $data[0] ) . '</a>';
    75101}
    76102
    77103list( $display_version ) = explode( '-', $wp_version );
     
    80106?>
    81107<div class="wrap about-wrap">
    82108
    83 <h1><?php printf( __( 'Welcome to WordPress %s' ), $display_version ); ?></h1>
     109<h1><?php printf( __( 'Welcome to WordPress %s' ), esc_html( $display_version ) ); ?></h1>
    84110
    85 <div class="about-text"><?php printf( __( 'Thank you for updating! WordPress %s helps you communicate and share, globally.' ), $display_version ); ?></div>
     111<div class="about-text"><?php printf( __( 'Thank you for updating! WordPress %s helps you communicate and share, globally.' ), esc_html( $display_version ) ); ?></div>
    86112
    87 <div class="wp-badge"><?php printf( __( 'Version %s' ), $display_version ); ?></div>
     113<div class="wp-badge"><?php printf( __( 'Version %s' ), esc_html( $display_version ) ); ?></div>
    88114
    89115<h2 class="nav-tab-wrapper">
    90116        <a href="about.php" class="nav-tab">
     
    101127$credits = wp_credits();
    102128
    103129if ( ! $credits ) {
    104         echo '<p class="about-description">' . sprintf( __( 'WordPress is created by a <a href="%1$s">worldwide team</a> of passionate individuals. <a href="%2$s">Get involved in WordPress</a>.' ),
     130        echo '<p class="about-description">' . esc_html( sprintf( __( 'WordPress is created by a <a href="%1$s">worldwide team</a> of passionate individuals. <a href="%2$s">Get involved in WordPress</a>.' ),
    105131                'https://wordpress.org/about/',
    106132                /* translators: Url to the codex documentation on contributing to WordPress used on the credits page */
    107                 __( 'https://codex.wordpress.org/Contributing_to_WordPress' ) ) . '</p>';
     133                __( 'https://codex.wordpress.org/Contributing_to_WordPress' ) ) ) . '</p>';
    108134        include( ABSPATH . 'wp-admin/admin-footer.php' );
    109135        exit;
    110136}
     
    124150                        $title = translate( $group_data['name'] );
    125151                }
    126152
    127                 echo '<h4 class="wp-people-group">' . $title . "</h4>\n";
     153                echo '<h4 class="wp-people-group">' . esc_html( $title ) . "</h4>\n";
    128154        }
    129155
    130156        if ( ! empty( $group_data['shuffle'] ) )
     
    141167                        break;
    142168                default:
    143169                        $compact = 'compact' == $group_data['type'];
    144                         $classes = 'wp-people-group ' . ( $compact ? 'compact' : '' );
     170                        $classes = esc_attr( 'wp-people-group ' . ( $compact ? 'compact' : '' ) );
    145171                        echo '<ul class="' . $classes . '" id="wp-people-group-' . $group_slug . '">' . "\n";
    146172                        foreach ( $group_data['data'] as $person_data ) {
    147                                 echo '<li class="wp-person" id="wp-person-' . $person_data[2] . '">' . "\n\t";
    148                                 echo '<a href="' . sprintf( $credits['data']['profiles'], $person_data[2] ) . '">';
     173                                echo '<li class="wp-person" id="wp-person-' . esc_attr( $person_data[2] ) . '">' . "\n\t";
     174                                echo '<a href="' . esc_url( sprintf( $credits['data']['profiles'], $person_data[2] ) ) . '">';
    149175                                $size = 'compact' == $group_data['type'] ? '30' : '60';
    150                                 echo '<img src="' . $gravatar . $person_data[1] . '?s=' . $size . '" srcset="' . $gravatar . $person_data[1] . '?s=' . $size * 2 . ' 2x" class="gravatar" alt="' . esc_attr( $person_data[0] ) . '" /></a>' . "\n\t";
    151                                 echo '<a class="web" href="' . sprintf( $credits['data']['profiles'], $person_data[2] ) . '">' . $person_data[0] . "</a>\n\t";
     176                                echo '<img src="' . esc_url( $gravatar . $person_data[1] . '?s=' . $size . '" srcset="' . $gravatar . $person_data[1] . '?s=' . $size * 2 ) . ' 2x" class="gravatar" alt="' . esc_attr( $person_data[0] ) . '" /></a>' . "\n\t";
     177                                echo '<a class="web" href="' . esc_url( sprintf( $credits['data']['profiles'], $person_data[2] ) ) . '">' . esc_html( $person_data[0] ) . "</a>\n\t";
    152178                                if ( ! $compact )
    153                                         echo '<span class="title">' . translate( $person_data[3] ) . "</span>\n";
     179                                        echo '<span class="title">' . esc_html( translate( $person_data[3] ) ) . "</span>\n";
    154180                                echo "</li>\n";
    155181                        }
    156182                        echo "</ul>\n";