diff options
author | Wing <steely.wing@gmail.com> | 2014-02-22 13:37:18 +0800 |
---|---|---|
committer | Wing <steely.wing@gmail.com> | 2014-02-22 13:37:18 +0800 |
commit | 3711f7b58bdb10dcdf3bf9622a03fa9dee459270 (patch) | |
tree | ffa22aa95bd9ac6e2ea172390964c8886a38190f /Http/RememberMe | |
parent | e8fa5b552b1c31fa90ee53a4571727da854f2654 (diff) | |
download | symfony-security-3711f7b58bdb10dcdf3bf9622a03fa9dee459270.zip symfony-security-3711f7b58bdb10dcdf3bf9622a03fa9dee459270.tar.gz symfony-security-3711f7b58bdb10dcdf3bf9622a03fa9dee459270.tar.bz2 |
use core StringUtils to compare hashes
Diffstat (limited to 'Http/RememberMe')
-rw-r--r-- | Http/RememberMe/TokenBasedRememberMeServices.php | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/Http/RememberMe/TokenBasedRememberMeServices.php b/Http/RememberMe/TokenBasedRememberMeServices.php index 571abbe..f3a39fe 100644 --- a/Http/RememberMe/TokenBasedRememberMeServices.php +++ b/Http/RememberMe/TokenBasedRememberMeServices.php @@ -17,6 +17,7 @@ use Symfony\Component\HttpFoundation\Response; use Symfony\Component\Security\Core\Authentication\Token\TokenInterface; use Symfony\Component\Security\Core\Exception\AuthenticationException; use Symfony\Component\Security\Core\User\UserInterface; +use Symfony\Component\Security\Core\Util\StringUtils; /** * Concrete implementation of the RememberMeServicesInterface providing @@ -77,16 +78,7 @@ class TokenBasedRememberMeServices extends AbstractRememberMeServices */ private function compareHashes($hash1, $hash2) { - if (strlen($hash1) !== $c = strlen($hash2)) { - return false; - } - - $result = 0; - for ($i = 0; $i < $c; $i++) { - $result |= ord($hash1[$i]) ^ ord($hash2[$i]); - } - - return 0 === $result; + return StringUtils::equals($hash1, $hash2); } /** |