这种密码散列和匹配方法的安全性如何？

Question

我从一系列帖子和一些先验知识中获取信息来实现以下哈希算法。然而，有很多关于哪些实现是安全的和不安全的讨论。我的方法如何衡量？安全吗？

public static function sha512($token,$cost = 50000,$salt = null) {
        $salt = ($salt == null) ? (generateToken(32)) : ($salt);
        $salt = '$6$rounds=' . $cost . '$' . $salt . ' $';
        return crypt($token, $salt);
}

public static function sha512Equals($token,$hash) {
    return (crypt($token,$hash) == $hash);
}


public static function generateToken($length,$characterPool = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ') {
    $token = '';
    $max = mb_strlen($characterPool);

    for ($i = 0;$i < $length;$i++){
        $token .= $characterPool[cryptorand(0,$max)];
    }

    return $token;
}

public static function cryptorand($min, $max) {
    $range = $max - $min;

    if ($range < 0) 
        return $min;

    $log = log($range, 2);
    $bytes = (int) ($log / 8) + 1; // length in bytes
    $bits = (int) $log + 1; // length in bits
    $filter = (int) (1 << $bits) - 1; // set all lower bits to 1

    do {
        $rnd = hexdec(bin2hex(openssl_random_pseudo_bytes($bytes)));
        $rnd = $rnd & $filter; // discard irrelevant bits
    } while ($rnd >= $range);

    return $min + $rnd;
}

那么这种方法安全吗？ PHP 中是否有更安全的方法用于散列令牌并稍后与令牌匹配？非常感谢任何批评。

Answer 1

不，因为您最终信任 crypt，并且您没有在 sha512Equals 中使用时间常数比较。

也可能存在特定于平台的问题：openssl_random_pseudo_bytes 不必是加密安全的。我不知道你怎么知道crypt 也使用 SHA-512。

cryptorand 中的计算略有偏差（例如，$log 的值恰好位于字节边界上），但幸运的是 do/while 循环对其进行了检查。

---

请改用password_hash 或password_verify 功能。