Merge "Use different varname for upgraded hash from original hash"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Thu, 17 Nov 2016 06:43:37 +0000 (06:43 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Thu, 17 Nov 2016 06:43:37 +0000 (06:43 +0000)
1  2 
includes/auth/LocalPasswordPrimaryAuthenticationProvider.php

@@@ -104,7 -104,7 +104,7 @@@ class LocalPasswordPrimaryAuthenticatio
                // The old hash format was just an md5 hex hash, with no type information
                if ( preg_match( '/^[0-9a-f]{32}$/', $row->user_password ) ) {
                        if ( $this->config->get( 'PasswordSalt' ) ) {
 -                              $row->user_password = ":A:{$row->user_id}:{$row->user_password}";
 +                              $row->user_password = ":B:{$row->user_id}:{$row->user_password}";
                        } else {
                                $row->user_password = ":A:{$row->user_password}";
                        }
  
                // @codeCoverageIgnoreStart
                if ( $this->getPasswordFactory()->needsUpdate( $pwhash ) ) {
-                       $pwhash = $this->getPasswordFactory()->newFromPlaintext( $req->password );
-                       \DeferredUpdates::addCallableUpdate( function () use ( $pwhash, $oldRow ) {
+                       $newHash = $this->getPasswordFactory()->newFromPlaintext( $req->password );
+                       \DeferredUpdates::addCallableUpdate( function () use ( $newHash, $oldRow ) {
                                $dbw = wfGetDB( DB_MASTER );
                                $dbw->update(
                                        'user',
-                                       [ 'user_password' => $pwhash->toString() ],
+                                       [ 'user_password' => $newHash->toString() ],
                                        [
                                                'user_id' => $oldRow->user_id,
                                                'user_password' => $oldRow->user_password