0
$register = "INSERT INTO wcf1_user (username, regIP, password, salt, email)
                    VALUES('$username', '$ip', SHA1(CONCAT(salt, SHA1(CONCAT(salt, $pwd)))), '$llamada', '$email')";

Hola, estoy intentando realizar un mysqli_query en php y ejecutar este código de sql en PHP, pero tengo un error al llamar la variable $pdw, necesito la contraseña en ese método, ¿Cómo puedo solucionarlo?

4
  • Si $pwd es una cadena debería ir entrecomillada. Cuidado con el valor de salt si la contraseña depende él, la columna debería ir antes.
    – Sal
    Commented el 16 may. 2022 a las 2:34
  • ¿Puedes explicar lo que intentas hacer aquí: SHA1(CONCAT(salt, SHA1(CONCAT(salt, $pwd))))? ¿Por qué dos CONCAT? Por otra parte, SHA1 no se considera seguro hoy día. Revisa las notas del MR: "Se han dado a conocer exploits para los algoritmos MD5 y SHA-1. Es posible que desee considerar el uso de otra función de cifrado unidireccional descrita en esta sección, como SHA2()." Desde PHP tú podrías trabajar con password_hash y password_verify. Además de esto, el código en sí mismo es vulnerable a inyección SQL.
    – A. Cedano
    Commented el 16 may. 2022 a las 9:33
  • @A.Cedano Estoy utilizando 2 concat por que así programé el registro de cuentas de esa manera dentro del servidor de juego, por otra parte, ¿Debería utilizar SHA2?
    – user264129
    Commented el 16 may. 2022 a las 9:42
  • Si la seguridad te importa sí. Lee el MR: "SHA2() puede considerarse criptográficamente más seguro que MD5() o SHA1().", pero, hay condiciones: "Esta función solo funciona si MySQL se ha configurado con soporte SSL".
    – A. Cedano
    Commented el 16 may. 2022 a las 9:46

2 respuestas 2

0

en php se concatena con PUNTO

$mivar1.$mivar2

o

$mivar1."un texto".$mivar2

o

$mivar1.'Otro Texto'.$mivar2
1
  • Intente utilizar esto y no funciona "INSERT INTO wcf1_user (username, regIP, password, salt, email) VALUES('$username', '$ip', SHA1(CONCAT(salt, SHA1(CONCAT(salt, '.$pwd.')))), '$llamada', '$email')";
    – user264129
    Commented el 16 may. 2022 a las 1:10
0

Tienes un error en tu codigo, no puedes usar una funcion dentro de un string...

la forma correcta de tu sentencia es:

$register = "INSERT INTO wcf1_user (username, regIP, password, salt, email)
                VALUES('$username', '$ip', '".SHA1($llamada . SHA1($llamada . $pwd))."', '$llamada', '$email')";

Ademas por cuestiones de legibilidad y mantenimiento, es recomendable generar primero la contraseña encriptada y despues guardarla..

$psw = SHA1($llamada . SHA1($llamada . $pwd));
$register = "INSERT INTO wcf1_user (username, regIP, password, salt, email)
                    VALUES('$username', '$ip', '$psw', '$llamada', '$email')";
2
  • ahora me dice esto: Uncaught Error: Call to undefined function CONCAT() in
    – user264129
    Commented el 16 may. 2022 a las 9:38
  • tienes toda la razon @user264129, al moverlo olvide convertir la concatenacion, ya hice la correccion del codigo, espero esto resuelva tu problema... saludos Commented el 19 may. 2022 a las 0:48

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.