WordPress.org

Make WordPress Core

Ticket #6266: 6266.diff

File 6266.diff, 4.8 KB (added by andy, 10 years ago)

let there be nonce renewal

  • wp-includes/js/autosave.js

     
    2424        if ( res && res.responses && res.responses.length ) {
    2525                message = res.responses[0].data; // The saved message or error.
    2626                // someone else is editing: disable autosave, set errors
    27                 if ( res.responses[0].supplemental && 'disable' == res.responses[0].supplemental['disable_autosave'] ) {
    28                         autosave = function() {};
    29                         res = { errors: true };
     27                if ( res.responses[0].supplemental ) {
     28                        if ( 'disable' == res.responses[0].supplemental['disable_autosave'] ) {
     29                                autosave = function() {};
     30                                res = { errors: true };
     31                        }
     32                        jQuery.each(res.responses[0].supplemental, function(selector, value) {
     33                                if ( selector.match(/^replace-/) ) {
     34                                        jQuery('#'+selector.replace('replace-', '')).val(value);
     35                                }
     36                        });
    3037                }
    3138
    3239                // if no errors: add preview link and slug UI
  • wp-includes/pluggable.php

     
    624624 * @param string $action Action nonce
    625625 * @param string $query_arg where to look for nonce in $_REQUEST (since 2.5)
    626626 */
    627 function check_admin_referer($action = -1, $query_arg = '_wpnonce' ) {
     627function check_admin_referer($action = -1, $query_arg = '_wpnonce') {
    628628        $adminurl = strtolower(get_option('siteurl')).'/wp-admin';
    629629        $referer = strtolower(wp_get_referer());
    630         if ( !wp_verify_nonce($_REQUEST[$query_arg], $action) &&
    631                 !(-1 == $action && strpos($referer, $adminurl) !== false)) {
     630        $result = wp_verify_nonce($_REQUEST[$query_arg], $action);
     631        if ( !$result && !(-1 == $action && strpos($referer, $adminurl) !== false) ) {
    632632                wp_nonce_ays($action);
    633633                die();
    634634        }
    635         do_action('check_admin_referer', $action);
     635        do_action('check_admin_referer', $action, $result);
     636        return $result;
    636637}endif;
    637638
    638639if ( !function_exists('check_ajax_referer') ) :
     
    644645 * @param string $action Action nonce
    645646 * @param string $query_arg where to look for nonce in $_REQUEST (since 2.5)
    646647 */
    647 function check_ajax_referer( $action = -1, $query_arg = false ) {
     648function check_ajax_referer( $action = -1, $query_arg = false, $die = true ) {
    648649        if ( $query_arg )
    649650                $nonce = $_REQUEST[$query_arg];
    650651        else
    651652                $nonce = $_REQUEST['_ajax_nonce'] ? $_REQUEST['_ajax_nonce'] : $_REQUEST['_wpnonce'];
    652653
    653         if ( !wp_verify_nonce( $nonce, $action ) )
     654        $result = wp_verify_nonce( $nonce, $action );
     655
     656        if ( $die && false == $result )
    654657                die('-1');
    655658
    656         do_action('check_ajax_referer');
     659        do_action('check_ajax_referer', $action, $result);
     660
     661        return $result;
    657662}
    658663endif;
    659664
     
    937942}
    938943endif;
    939944
     945if ( !function_exists('wp_nonce_tick') ) :
     946/**
     947 * wp_nonce_tick() - Get the time-dependent variable for nonce creation
     948 *
     949 * A nonce has a lifespan of two ticks. Nonces in their second tick may be updated, e.g. by autosave.
     950 *
     951 * @since 2.5
     952 *
     953 * @return int
     954 */
     955function wp_nonce_tick() {
     956        $nonce_life = apply_filters('nonce_life', 86400) / 2;
     957
     958        return ceil(time() / ( $nonce_life / 2 ));
     959}
     960endif;
     961
    940962if ( !function_exists('wp_verify_nonce') ) :
    941963/**
    942964 * wp_verify_nonce() - Verify that correct nonce was used with time limit
     
    954976        $user = wp_get_current_user();
    955977        $uid = (int) $user->id;
    956978
    957         $i = ceil(time() / 43200);
     979        $i = wp_nonce_tick();
    958980
    959         //Allow for expanding range, but only do one check if we can
    960         if( substr(wp_hash($i . $action . $uid), -12, 10) == $nonce || substr(wp_hash(($i - 1) . $action . $uid), -12, 10) == $nonce )
    961                 return true;
     981        // Nonce generated 0-12 hours ago
     982        if ( substr(wp_hash($i . $action . $uid), -12, 10) == $nonce )
     983                return 1;
     984        // Nonce generated 12-24 hours ago
     985        if ( substr(wp_hash(($i - 1) . $action . $uid), -12, 10) == $nonce )
     986                return 2;
     987        // Invalid nonce
    962988        return false;
    963989}
    964990endif;
     
    9761002        $user = wp_get_current_user();
    9771003        $uid = (int) $user->id;
    9781004
    979         $i = ceil(time() / 43200);
     1005        $i = wp_nonce_tick();
    9801006
    9811007        return substr(wp_hash($i . $action . $uid), -12, 10);
    9821008}
  • wp-admin/admin-ajax.php

     
    460460        $x->send();
    461461        break;
    462462case 'autosave' : // The name of this action is hardcoded in edit_post()
    463         check_ajax_referer( 'autosave', 'autosavenonce' );
     463        $nonce_age = check_ajax_referer( 'autosave', 'autosavenonce');
    464464        global $current_user;
    465465
    466466        $_POST['post_status'] = 'draft';
     
    520520        if ( $do_lock && $id && is_numeric($id) )
    521521                wp_set_post_lock( $id );
    522522
     523        if ( $nonce_age == 2 )
     524                $supplemental['replace-autosavenonce'] = wp_create_nonce('autosave');
     525
    523526        $x = new WP_Ajax_Response( array(
    524527                'what' => 'autosave',
    525528                'id' => $id,