¿Cuál es la mejor manera de comparar cadenas hash? (PHP)

Question

¿Debo usar if(strcmp(md5($string),$hash)==0) o if(md5($string)==$hash)

Answer 1

== se ha demostrado por otros usuarios aquí para ser extremadamente fiable. Debería usar strcmp() en su lugar.

Una tercera opción, si realmente desea utilizar un operador de comparación, es el uso de === que no realiza tipo de coacción de ninguna clase, y por lo tanto conserva tipos y valores para los fines de comparación.

Answer 2

Creo que if(md5($string) == $hash) es mejor, porque solo tiene una comparación en lugar de 2 (stcmp & ==).

md5 genera solo ascii-chars que no necesitan una comparación binaria segura.

Answer 3

Si compara cadenas, use strcmp o ===. La gente prefiere === porque strcmp puede ser confuso (devuelve 0 en caso de éxito, wat).

Debe usar ===, no ==. == convertirá ambos operandos en enteros si se pueden interpretar como tales, y debido a que un hash MD5 no cabe en un entero, se truncarán alrededor de la mitad. Por lo tanto, solo las primeras mitades de los hashes deben ser iguales. Ver http://phpsadness.com/sad/47.

Si usa contraseñas hash, considere usar un algoritmo hash lento y fuerte como PBKDF2, en lugar de MD5.

Answer 4

Debe tener mucho cuidado al comparar los hashes directamente para cosas como la autenticación ya que podría estar abriendo una ventana a un ataque de temporización.

Aunque suena muy contradictorio, debería hacer una comparación completa de la cadena, evitando cualquier optimización (es decir, salir temprano si un personaje es diferente).

Éstos son algunos enlaces sobre el problema:

• https://wiki.php.net/rfc/timing_attack
• http://codahale.com/a-lesson-in-timing-attacks/
• http://carlos.bueno.org/2011/10/timing.html

Y aquí están algunas ideas con el fin de solucionarlo:

• El más rápido y fácil: simplemente use hash_equals() si está disponible (PHP 5.6 y superior).
• https://github.com/symfony/polyfill-php56/blob/master/Php56.php (hash_equals() polyfill)
• https://github.com/zendframework/zend-crypt/blob/master/src/Utils.php (compareStrings())

Answer 5

Si estás usando nada más nuevo que PHP 5.6 (e incluyendo) se debe utilizar timing attack safe string comparison función.

if (hash_equals($expected, $correct)) { 

} 

(Si estás en PHP 5.5 o anterior, see here for equivalents.)

Answer 6

En realidad se debe utilizar para este password_verify y también utilizar todas las demás funciones password_*. Están disponibles en PHP> = 5.5.0.

Como una alternativa, puede usar this polyfill. Actualmente funciona con PHP> = 5.3.7.

Y si realmente no puedes/quieres usar esto todavía hay hash_equals (y polyfills para eso) como @MM. ya dicho.