MySQLi déclaration préparée se plaint que "seules les variables doivent être passés par référence"

Question

code:

$stmt->bind_param("s", md5($input['user'] . $config['salt'])); 

Erreur PHP Message:

Seules les variables doivent être transmises par référence

J'ai travaillé sur ce projet mais je suis coincé maintenant. Je suis nouveau à PHP. Que faire?

Answer 1

Merci d'utiliser les instructions préparées MySQLi! Ils sont une douleur, mais ça vaut le coup.

bind_param prend les valeurs par référence. Il le fait en regardant la variable que vous passez et en pointant vers les entrailles directement.

Dans votre appel, vous renvoyez le résultat de chaîne d'un appel de fonction - md5 dans ce cas. Parce qu'il n'y a pas de variable impliquée, il n'y a pas d'entrailles à pointer. PHP se plaint de ne pas pouvoir transmettre les données par référence.

Vous devrez coller le résultat de l'appel de fonction dans une variable, puis passer cette variable dans la liaison à la place.

BIG FAT AVERTISSEMENT!md5 est pas un hachage sécurisé plus longtemps, et devrait pas être utilisé pour stocker les mots de passe. Lorsque vous avez la chance, vous devriez mettre à jour vers un meilleur format de hachage, tel que bcrypt, PBKDF2, scrypt, etc.

Answer 2

Chaque paramètre (mais le premier) du bind_param method doit être une variable et non comme dans votre cas, une valeur de retour de fonction. Seules les variables peuvent être passed by reference.

Avec cela à l'esprit, vous pouvez facilement changer le code pour se débarrasser du message d'erreur:

$input['hash'] = md5($input['pass'] . $config['salt']); 
$stmt->bind_param("ss", $input['user'], $input['hash']);