WordPress.org
Get WordPress

Make WordPress Core

Changeset 24480 for trunk


Ignore:
Timestamp:
06/21/2013 06:07:47 AM (13 years ago)
Author:
nacin
Message:

Better validation of the URL used in core HTTP requests.

Location:
trunk
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-admin/includes/class-wp-importer.php

    r22108 r24480  
    184184                $headers = array();
    185185                $args = array();
     186                $args['reject_unsafe_urls'] = true;
    186187                if ( true === $head )
    187188                        $args['method'] = 'HEAD';

  • trunk/wp-admin/includes/file.php

    r24463 r24480  
    498498                return new WP_Error('http_no_file', __('Could not create Temporary file.'));
    499499
    500         $response = wp_remote_get( $url, array( 'timeout' => $timeout, 'stream' => true, 'filename' => $tmpfname ) );
     500        $response = wp_remote_get( $url, array( 'timeout' => $timeout, 'stream' => true, 'filename' => $tmpfname, 'reject_unsafe_urls' => true ) );
    501501
    502502        if ( is_wp_error( $response ) ) {

  • trunk/wp-includes/class-feed.php

    r22811 r24480  
    6767
    6868                if ( preg_match('/^http(s)?:\/\//i', $url) ) {
    69                         $args = array( 'timeout' => $this->timeout, 'redirection' => $this->redirects);
     69                        $args = array(
     70                                'timeout' => $this->timeout,
     71                                'redirection' => $this->redirects,
     72                                'reject_unsafe_urls' => true,
     73                        );
    7074
    7175                        if ( !empty($this->headers) )
    … …  
    8690                        }
    8791                } else {
    88                         if ( ! file_exists($url) || ( ! $this->body = file_get_contents($url) ) ) {
    89                                 $this->error = 'file_get_contents could not read the file';
    90                                 $this->success = false;
    91                         }
     92                        $this->error = '';
     93                        $this->success = false;
    9294                }
    9395        }

  • trunk/wp-includes/class-http.php

    r24303 r24480  
    8787                        'redirection' => apply_filters( 'http_request_redirection_count', 5),
    8888                        'httpversion' => apply_filters( 'http_request_version', '1.0'),
    89                         'user-agent' => apply_filters( 'http_headers_useragent', 'WordPress/' . $wp_version . '; ' . get_bloginfo( 'url' )  ),
     89                        'user-agent' => apply_filters( 'http_headers_useragent', 'WordPress/' . $wp_version . '; ' . get_bloginfo( 'url' ) ),
     90                        'reject_unsafe_urls' => apply_filters( 'http_request_reject_unsafe_urls', false ),
    9091                        'blocking' => true,
    9192                        'headers' => array(),
    … …  
    119120                        return $pre;
    120121
    121                 $arrURL = parse_url( $url );
     122                if ( $r['reject_unsafe_urls'] )
     123                        $url = wp_http_validate_url( $url );
     124                $url = wp_kses_bad_protocol( $url, array( 'http', 'https', 'ssl' ) );
     125
     126                $arrURL = @parse_url( $url );
    122127
    123128                if ( empty( $url ) || empty( $arrURL['scheme'] ) )
    … …  
    11471152                // bug #17490 with redirected POST requests, so handle redirections outside Curl.
    11481153                curl_setopt( $handle, CURLOPT_FOLLOWLOCATION, false );
     1154                if ( defined( 'CURLOPT_PROTOCOLS' ) ) // PHP 5.2.10 / cURL 7.19.4
     1155                        curl_setopt( $handle, CURLOPT_PROTOCOLS, CURLPROTO_HTTP | CURLPROTO_HTTPS );
    11491156
    11501157                switch ( $r['method'] ) {

  • trunk/wp-includes/class-oembed.php

    r24470 r24480  
    114114
    115115                // Fetch URL content
    116                 if ( $html = wp_remote_retrieve_body( wp_remote_get( $url ) ) ) {
     116                if ( $html = wp_remote_retrieve_body( wp_remote_get( $url, array( 'reject_unsafe_urls' => true ) ) ) ) {
    117117
    118118                        // <link> types that contain oEmbed provider URLs
    … …  
    196196        function _fetch_with_format( $provider_url_with_args, $format ) {
    197197                $provider_url_with_args = add_query_arg( 'format', $format, $provider_url_with_args );
    198                 $response = wp_remote_get( $provider_url_with_args );
     198                $response = wp_remote_get( $provider_url_with_args, array( 'reject_unsafe_urls' => true ) );
    199199                if ( 501 == wp_remote_retrieve_response_code( $response ) )
    200200                        return new WP_Error( 'not-implemented' );

  • trunk/wp-includes/class-wp-xmlrpc-server.php

    r24382 r24480  
    53975397
    53985398                // Let's check the remote site
    5399                 $linea = wp_remote_retrieve_body( wp_remote_get( $pagelinkedfrom, array( 'timeout' => 10, 'redirection' => 0 ) ) );
     5399                $linea = wp_remote_retrieve_body( wp_remote_get( $pagelinkedfrom, array( 'timeout' => 10, 'redirection' => 0, 'reject_unsafe_urls' => true ) ) );
     5400
    54005401                if ( !$linea )
    54015402                        return $this->pingback_error( 16, __( 'The source URL does not exist.' ) );

  • trunk/wp-includes/comment.php

    r24301 r24480  
    16591659                return false;
    16601660
    1661         $response = wp_remote_head( $url, array( 'timeout' => 2, 'httpversion' => '1.0' ) );
     1661        $response = wp_remote_head( $url, array( 'timeout' => 2, 'httpversion' => '1.0', 'reject_unsafe_urls' => true ) );
    16621662
    16631663        if ( is_wp_error( $response ) )
    … …  
    16721672
    16731673        // Now do a GET since we're going to look in the html headers (and we're sure it's not a binary file)
    1674         $response = wp_remote_get( $url, array( 'timeout' => 2, 'httpversion' => '1.0' ) );
     1674        $response = wp_remote_get( $url, array( 'timeout' => 2, 'httpversion' => '1.0', 'reject_unsafe_urls' => true ) );
    16751675
    16761676        if ( is_wp_error( $response ) )
    … …  
    19071907        $options = array();
    19081908        $options['timeout'] = 4;
     1909        $options['reject_unsafe_urls'] = true;
    19091910        $options['body'] = array(
    19101911                'title' => $title,
    … …  
    19541955 *
    19551956 * @since 3.5.1
     1957 * @see wp_http_validate_url()
    19561958 *
    19571959 * @param string $source_uri
    … …  
    19591961 */
    19601962function pingback_ping_source_uri( $source_uri ) {
    1961         $uri = esc_url_raw( $source_uri, array( 'http', 'https' ) );
    1962         if ( ! $uri )
    1963                 return '';
    1964 
    1965         $parsed_url = @parse_url( $uri );
    1966         if ( ! $parsed_url )
    1967                 return '';
    1968 
    1969         if ( isset( $parsed_url['user'] ) || isset( $parsed_url['pass'] ) )
    1970                 return '';
    1971 
    1972         if ( false !== strpos( $parsed_url['host'], ':' ) )
    1973                 return '';
    1974 
    1975         $parsed_home = @parse_url( get_option( 'home' ) );
    1976 
    1977         $same_host = strtolower( $parsed_home['host'] ) === strtolower( $parsed_url['host'] );
    1978 
    1979         if ( ! $same_host ) {
    1980                 $host = trim( $parsed_url['host'], '.' );
    1981                 if ( preg_match( '#^\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3}$#', $host ) ) {
    1982                         $ip = $host;
    1983                 } else {
    1984                         $ip = gethostbyname( $host );
    1985                         if ( $ip === $host ) // Error condition for gethostbyname()
    1986                                 $ip = false;
    1987                 }
    1988                 if ( $ip ) {
    1989                         if ( '127.0.0.1' === $ip )
    1990                                 return '';
    1991                         $parts = array_map( 'intval', explode( '.', $ip ) );
    1992                         if ( 10 === $parts[0] )
    1993                                 return '';
    1994                         if ( 172 === $parts[0] && 16 <= $parts[1] && 31 >= $parts[1] )
    1995                                 return '';
    1996                         if ( 192 === $parts[0] && 168 === $parts[1] )
    1997                                 return '';
    1998                 }
    1999         }
    2000 
    2001         if ( empty( $parsed_url['port'] ) )
    2002                 return $uri;
    2003 
    2004         $port = $parsed_url['port'];
    2005         if ( 80 === $port || 443 === $port || 8080 === $port )
    2006                 return $uri;
    2007 
    2008         if ( $parsed_home && $same_host && $parsed_home['port'] === $port )
    2009                 return $uri;
    2010 
    2011         return '';
     1963        return (string) wp_http_validate_url( $source_uri );
    20121964}
    20131965

  • trunk/wp-includes/functions.php

    r24444 r24480  
    497497        $options = array();
    498498        $options['redirection'] = 5;
     499        $options['reject_unsafe_urls'] = true;
    499500
    500501        if ( false == $file_path )
    … …  
    544545                _deprecated_argument( __FUNCTION__, '2.7' );
    545546
    546         $response = wp_remote_head( $url );
     547        $response = wp_remote_head( $url, array( 'reject_unsafe_urls' => true ) );
    547548
    548549        if ( is_wp_error( $response ) )
    … …  
    759760        $options = array();
    760761        $options['timeout'] = 10;
     762        $options['reject_unsafe_urls'] = true;
    761763
    762764        $response = wp_remote_get( $uri, $options );

  • trunk/wp-includes/http.php

    r24250 r24480  
    331331        return false;
    332332}
     333
     334/**
     335 * Validate a URL for safe use in the HTTP API.
     336 *
     337 * @since 3.5.2
     338 *
     339 * @return mixed URL or false on failure.
     340 */
     341function wp_http_validate_url( $url ) {
     342        $url = esc_url_raw( $url, array( 'http', 'https' ) );
     343        if ( ! $url )
     344                return false;
     345
     346        $parsed_url = @parse_url( $url );
     347        if ( ! $parsed_url )
     348                return false;
     349
     350        if ( isset( $parsed_url['user'] ) || isset( $parsed_url['pass'] ) )
     351                return false;
     352
     353        if ( false !== strpos( $parsed_url['host'], ':' ) )
     354                return false;
     355
     356        $parsed_home = @parse_url( get_option( 'home' ) );
     357
     358        $same_host = strtolower( $parsed_home['host'] ) === strtolower( $parsed_url['host'] );
     359
     360        if ( ! $same_host ) {
     361                $host = trim( $parsed_url['host'], '.' );
     362                if ( preg_match( '#^\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3}$#', $host ) ) {
     363                        $ip = $host;
     364                } else {
     365                        $ip = gethostbyname( $host );
     366                        if ( $ip === $host ) // Error condition for gethostbyname()
     367                                $ip = false;
     368                }
     369                if ( $ip ) {
     370                        if ( '127.0.0.1' === $ip )
     371                                return false;
     372                        $parts = array_map( 'intval', explode( '.', $ip ) );
     373                        if ( 10 === $parts[0] )
     374                                return false;
     375                        if ( 172 === $parts[0] && 16 <= $parts[1] && 31 >= $parts[1] )
     376                                return false;
     377                        if ( 192 === $parts[0] && 168 === $parts[1] )
     378                                return false;
     379                }
     380        }
     381
     382        if ( empty( $parsed_url['port'] ) )
     383                return $url;
     384
     385        $port = $parsed_url['port'];
     386        if ( 80 === $port || 443 === $port || 8080 === $port )
     387                return $url;
     388
     389        if ( $parsed_home && $same_host && $parsed_home['port'] === $port )
     390                return $url;
     391
     392        return false;
     393}

  • trunk/wp-includes/rss.php

    r23191 r24480  
    537537 */
    538538function _fetch_remote_file($url, $headers = "" ) {
    539         $resp = wp_remote_request($url, array('headers' => $headers, 'timeout' => MAGPIE_FETCH_TIME_OUT));
     539        $resp = wp_remote_request($url, array('headers' => $headers, 'timeout' => MAGPIE_FETCH_TIME_OUT, 'reject_unsafe_urls' => true ));
    540540        if ( is_wp_error($resp) ) {
    541541                $error = array_shift($resp->errors);
Note: See TracChangeset for help on using the changeset viewer.