0

Necesito guardar un password (que introduce un usuario mediante un formulario php) en una tabla de una base de datos. Tengo el siguiente código:

else {               
  $query = "UPDATE datos 
            SET password = '".md5(md5($_POST['password']))."' 
            WHERE email = '".$_POST['email']."'";
  mysqli_query($con, $query);                                                   
}  

Pero no se ejecuta este $query, me imagino que debo tener algo mal escrito, lo saqué de un ejemplo que vi, pero no me funciona. Si alguien me puede ayudar, se lo agradecería mucho.

Saludos cordiales :)

6
  • podrias agrege el codigo completo de PHP?
    – simon
    Commented el 5 jul. 2017 a las 0:49
  • Este código es vulnerable a ataques de inyección SQL. Deberías usar sentencias preparadas en lugar de concatenar valores.
    – Alvaro Montoro
    Commented el 5 jul. 2017 a las 1:22
  • Te dejo un posible respuesta a tu duda es.stackoverflow.com/questions/33420/…
    – D.Bulten
    Commented el 5 jul. 2017 a las 11:31
  • 1
    Muchas gracias Bulten, estoy leyendo esa respuesta y es perfecta para mi caso. Te lo agradezco mucho :)
    – Olga L. V.
    Commented el 5 jul. 2017 a las 16:01
  • podriamos decir que tu pregunta es un duplicado de esa?? o esta relacionada???
    – gbianchi
    Commented el 5 jul. 2017 a las 16:11

2 respuestas 2

2

Estás usando una sintaxis incorrecta para este tipo de operaciones y un tipo de cifrado vulnerable.

Te dejo por aquí una pagina para que le eches un ojo. Las ideas esenciales de ese enlace son:

  • No almacenes las contraseñas en texto plano.
  • No intentes crear tu propio sistema de seguridad de contraseña.
  • No "encriptes" contraseñas (porque el proceso es reversible).
  • No uses MD5.
  • No uses una semilla (sal/salt) común para todo el sitio.
  • Lo que deberías hacer:
    • Usa una función criptográfica de cifrado fuerte como bcrypt (mira crypt en PHP).
    • Usa una semilla diferente para cada contraseña.
    • Usa un algoritmo de hashing lento para hacer que los ataques de fuerza bruta prácticamente imposibles.
    • Para seguridad extra, regenera el hash cada vez que el usuario hace login.
1
  • Muchísimas gracias por el enlace. La información es muy buena y ya estoy probando.
    – Olga L. V.
    Commented el 5 jul. 2017 a las 15:48
0

Ah claro, habrás olvidado el punto y coma de la sentencia SQL, independientemente del punto y coma de PHP.

WHERE email = '".$_POST['email']."'";

POR

WHERE email = '".$_POST['email']."';";

Explicación? Cuando ejecutas un query en la base de datos y olvidas el punto y coma al final, la consola se queda esperando a que sigas escribiendo instrucciones, es hasta que lee el ; (punto y coma) que ya ejecuta lo que escribiste y te arroja el resultado o error.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.