Bitte überprüfen Sie schnell diese PHP + SALT-Implementierung - funktioniert nicht?

Question

Aufbauend auf Tutorials, um ein grundlegendes Benutzeranmeldung + Anmeldesystem mit Salz zu implementieren. Im Moment benutze ich dies für die Anmeldestufe:

define('SALT_LENGTH', 9); 

function generateHash($plainText, $salt = null) 
{ 
    if ($salt === null) 
    { 
     $salt = substr(md5(uniqid(rand(), true)), 0, SALT_LENGTH); 
    } 
    else 
    { 
     $salt = substr($salt, 0, SALT_LENGTH); 
    } 

    return $salt . sha1($salt . $plainText); 
} 

$newpass = generateHash($_POST['newpass']); 

followed by: 

$sql = "INSERT INTO user SET 
       userid = '$_POST[newid]', 
       password = PASSWORD('$newpass'), ... etc" 

Das funktioniert gut.

Ich mag jetzt eingegebene Passwort vergleichen für Gleichheit (in einer separaten Zugriffskontrolldatei) zu überprüfen:

define('SALT_LENGTH', 9); 

function generateHash($plainText, $salt) 
{ 

    $salt = substr($salt, 0, SALT_LENGTH); 

    return $salt . sha1($salt . $plainText); 
} 

$sql = "SELECT password FROM user WHERE 
     userid = '$uid'"; 
$result = mysql_query($sql); 
$row = mysql_fetch_row($result); 


$comparepwd = generateHash($pwd, $row['password']); 


if (mysql_num_rows($result) == 0 || $comparepwd != $row['password']) { 

//access denied, unset session variables 
} 

Grundsätzlich glaube ich, das sollte funktionieren. Ich bin ziemlich neu mit PHP/MySQL, also wäre ich sehr dankbar, wenn Sie darüber beraten könnten, warum es nicht funktioniert. Vielen Dank!

EDIT: Nur realisiert, ist es, weil

INSERT INTO user SET 
        userid = '$_POST[newid]', 
        password = PASSWORD('$newpass') 

das Passwort ein ('$ newpass') macht weiter MySQL Häsing?

Answer 1

Ja, die Passwort-Funktion ist ein Einweg-Hash und Sie sollten es nicht wirklich verwenden!

http://dev.mysql.com/doc/refman/5.0/en/encryption-functions.html#function_password