funkervogt
Goto Top

Verändert ein SQL Befehl einen MD5 Wert bzw. kuriosum im MD5 Vergleich!

Hallo Ihr PHP-Gurus!

Seit 3 Tagen sitz ich jetzt an einem, ich denke mal später
dann sehr trivialen, Problem.

Situation:

Ich hab eine HTML mit Login Nutzer und Passwort, deren Inhalte ich per Post an eine
.php übergebe.

HTML:
<div style="position:absolute; bottom:5px; left:90px; font-size: 100%; color:#FFFFFF; font-family: AmericanText BT, Comic, cursive, Arial">  
 <form action="HSRV2.php" method="POST" >  
  <p>N i c k n a m e:<br /><input name="Name" type="text" size="30" maxlength="30" style="font-family: AmericanText BT" />  
  <p>P a s s w o r t:<br /><input name="Pass" type="Password" size="30" maxlength="30" style="font-family: AmericanText BT" />  
  <p><input type="submit" value="L o g  I n" style="font-family: AmericanText BT, Comic, cursive, Arial"/></p>  
</div>

Im php Script, wird das Passwort in einen MD5 Wert verändert.
Dann sollte dieser MD5-Wert mit einem vorhandenen MD5-Wert aus der
MySQL DB verglichen werden! Macht er auch

aber:

Der Vergleich sagt immer: <> (ungleich) Bis auf in einem
Fall! Wenn ich ein bestimmtes Kennwort in alle User in der DB eintrage,
dann geht es kurioser Weise! Ich bin irgendwie ratlos!

Das .php Script:
<?php
// Kontakt Datenbank
$DBverb = mysql_connect("127.0.0.1","Hilfe","MD5MD5MD5") or die ("Log In momentan nicht m&ouml;glich, wir arbeiten daran!");  
mysql_select_db("hsrv", $DBverb);  

$password = $_POST['Pass'];  
$name = $_POST['Name'];  

$ergebnis = mysql_query("SELECT `Passwort` FROM `user` WHERE `Nickname` = '$name'");  

if ($ergebnis == md5($_POST['Pass']))  
{
                echo '<script language ="JavaScript">  
                <!--
                document.location.href="HSRV2.html";  
                // -->
                </script>';  
}
else {
        echo '<script language ="JavaScript">  
        <!--
        document.location.href="false.html";  
        // -->
        </script>';  
}
?>

Kann doch eigentlich nicht an der DB liegen, wenn es mit einem bestimmten Passwort geht! Oder?

[Edit Biber] Code formatiert. [/Edit]

Content-Key: 175081

Url: https://administrator.de/contentid/175081

Printed on: April 20, 2024 at 04:04 o'clock

Member: Arano
Arano Oct 21, 2011 at 13:49:12 (UTC)
Goto Top
Hallo FunkerVogt,

Bitte verwende doch die -Tags aus der Formatierungshilfe (roter Link über dem Eingabefeld)


$ergebnis = mysql_query("SELECT `Passwort` FROM `user` WHERE `Nickname` = '$name'");

if ($ergebnis == md5($_POST['Pass']))
Hier liegt dein Hund begraben, mysql_query() liefert etwas anders zurück als du erwartest !
Schau doch noch einmal im PHP-Manual - mysql_query() nach.

Tipp:
Ruhig während des Programmierens mal ein paar Variablen per echo ausgeben lassen und deren Inhalt kontrollieren.
siehe dazu im Manual auch nach den beiden Funktionen print_r() und var_dump()

Und weil du gerade im Manual unterwegs bist, schau auch gleich noch einmal nach header(), das funktioniert auch wen der Besucher Javascript deaktiviert hat ;)


~Arano

Edit:
Da sehe ich gerade noch ein kleinen Tipp.
Dein Query solltest du noch um einem LIMIT 1 ergänzen, dann braucht die Datenbank nicht noch alle Einträge __nach dem Treffer__ kontrollieren die weiss ja nicht, das es nur einen Benutzer mit dem Namen geben kann.

weiter Edit...
Du kannst auch der Datenbank den Passwort-/MD5-Vergleich überlassen, dann brauchst du in PHP nur noch auswerten ob es einen Treffer gab oder nicht.
Member: FunkerVogt
FunkerVogt Oct 21, 2011 at 15:33:42 (UTC)
Goto Top
Danke für die Tips, nehm ich mir zu Herzen!

Die Lösung:


$ergebnis = mysql_query("SELECT `Passwort` FROM `user` WHERE `Nickname` = '$name'");
$row = mysql_fetch_row($ergebnis);
$ergebnisRes = ($row);

if ($ergebnisRes == md5($password))


Formatierung hier im Forum auch gleich gelernt! face-smile

Vielen Dank!
Member: Arano
Arano Oct 22, 2011 at 10:13:46 (UTC)
Goto Top
Moin moin,

hat ja super geklappt !

if ($ergebnisRes == md5($password))
Wie du das jetzt vielleicht selber siehst, ist dieser Bedingung nicht gleich anzusehen was sie überhaupt vergleicht. Jetzt weisst du das noch, aber später kann es ganz schön schwer sein, den eigentlichen Sinn dahinter wieder zu erkennen. (oder dritte Personen z.B. in Foren)
Für Variablen sollten Namen gewählt werden, von denen man auf den Inhalt schließen kann.
<?php
$strQuerySelectUser = "SELECT `Passwort` FROM `user` WHERE `Nickname` = '$name'";  
$resSelectUser = mysql_query($strQuerySelectUser);

// und dann

$arrUser = mysql_fetch_row($resSelectUser);
$strMD5Pass = $arrUser;
if ($strMD5Pass == md5($password))

// oder

$arrUser = mysql_fetch_assoc($resSelectUser);
if ($arrUser['Passwort'] == md5($password))  
?>

Und als weitere Tipps:
  • Was wenn die Anfrage doch nicht an den Server gesendet werden kann mysql_query() (z.B. Weil der Server nicht mehr erreichbar ist, oder die Tabelle Fehler aufweist, ...) PHP-Manual - is_ressource()
  • Was wenn es den User __nicht gibt__ und so nur eine leere Ergebnistabelle als Ressourcenkennung zurückgegeben wird ? (z.B. Ein Tippfehler beim Benutzernamen) PHP-Manual - mysql_num_rows()


Schönes Wochenende
~Arano