Changeset 44780

Timestamp:

02/28/2019 02:38:27 PM (6 years ago)

Author:

SergeyBiryukov

Message:

Users: Replace raw SQL query in check_password_reset_key() with get_user_by().

Props davidbaumwald, iworks, spacedmonkey.
Fixes #45845.

File:

• trunk/src/wp-includes/user.php (modified) (4 diffs)

trunk/src/wp-includes/user.php

@@ -2321 +2321 @@
     }
 
-    $row = $wpdb->get_row( $wpdb->prepare( "SELECT ID, user_activation_key FROM $wpdb->users WHERE user_login = %s", $login ) );
-    if ( ! $row ) {
+    $user = get_user_by( 'login', $login );
+
+    if ( ! $user ) {
         return new WP_Error( 'invalid_key', __( 'Invalid key' ) );
     }
@@ -2340 +2341 @@
     $expiration_duration = apply_filters( 'password_reset_expiration', DAY_IN_SECONDS );
 
-    if ( false !== strpos( $row->user_activation_key, ':' ) ) {
-        list( $pass_request_time, $pass_key ) = explode( ':', $row->user_activation_key, 2 );
+    if ( false !== strpos( $user->user_activation_key, ':' ) ) {
+        list( $pass_request_time, $pass_key ) = explode( ':', $user->user_activation_key, 2 );
         $expiration_time                      = $pass_request_time + $expiration_duration;
     } else {
-        $pass_key        = $row->user_activation_key;
+        $pass_key        = $user->user_activation_key;
         $expiration_time = false;
     }
@@ -2355 +2356 @@
 
     if ( $hash_is_correct && $expiration_time && time() < $expiration_time ) {
-        return get_userdata( $row->ID );
+        return $user;
     } elseif ( $hash_is_correct && $expiration_time ) {
         // Key has an expiration time that's passed
@@ -2361 +2362 @@
     }
 
-    if ( hash_equals( $row->user_activation_key, $key ) || ( $hash_is_correct && ! $expiration_time ) ) {
+    if ( hash_equals( $user->user_activation_key, $key ) || ( $hash_is_correct && ! $expiration_time ) ) {
         $return  = new WP_Error( 'expired_key', __( 'Invalid key' ) );
-        $user_id = $row->ID;
+        $user_id = $user->ID;
 
         /**