"Ihr Passwort ist falsch" - Login defekt

KageMurai · 20. August 2007, 18:29

Hallo,

tut mir leid, aber ich sitze jetzt seit ner halben Stunde davor und weiß nciht, was ich falsch mache.

Ich lese mit

Quellcode

$ergebnis = mysql_query("SELECT nick, pass FROM user WHERE nick = '".$_POST['username']."'") or die('Auswählen fehlgeschlagen!');

Das Passwort des Users aus.

Warum gibt dann diese Abfrage aber immer "Ihr passwort ist falsch"?

Quellcode

if (md5($_POST['pw'])==$ergebnis['pass']) {

KageMurai, genervt

CookIe · 20. August 2007, 18:37

Schonmal mit:

Quellcode

if($_POST['pw']) == md5($ergebnis['pass'])) {

.. probiert?

//Klammer vergessen, is etz dabei!

SeBa · 20. August 2007, 18:38

Fehlt da nicht ein msql_fetch_assoc()? Oder hast du es nur nicht hingeschrieben. Deine Abfrage ist übrigens anfällig gegen MySQL-Injections.

CookIe dein Ansatz würde bedeuten der User gibt seinen MD5-Hash ins Passwortfeld ein und in der DB steht das Passwort im Klartext.

KageMurai · 20. August 2007, 18:42

Nein, mit dem Code klappt es nicht!

@SeBa: Wie sieht denn deiner Meinung nach das dazugehörige mysql_fetch_array aus?

Danke!

kageMurai

CookIe · 20. August 2007, 18:43

Ich bin einfach davon ausgegangen dass er alles andere hat.

Eigtl. ist es ja so:

Quellcode

$qry = mysql_query("..abfrage hier");
$ergebnis = mysql_fetch_array($qry);
if(md5($_POST['pw']) == $ergebnis['pass']) {

So müsste es eigtl. funktionieren .. wenn ich mich nicht täusch'.

// @SeBa:
Habe ich auch gesehen, sry.

KageMurai · 20. August 2007, 18:45

Nein, tut er bei mir komischerweise auch nicht (Ja, die Tabelle gibt es mit allen Einrägen)

Quellcode

if ($_POST['name'] == "") {
echo "Bitte geben Sie einen Namen ein!";
}
elseif ($_POST['pw'] == "") {
echo "Bitte geben Sie ein Passwort ein!";
}
else {
$connection=mysql_connect($server,$user,$pass) or die (mysql_error());
mysql_select_db("nedias",$connection) or die (mysql_error());
$user_pass = md5($_POST['pass']);
$query = mysql_query("SELECT pass FROM user WHERE nick = '".$_POST['username']."'") or die('Auswählen fehlgeschlagen!');
$ergebnis=mysql_fetch_array($query);
if($_POST['pw'] == md5($ergebnis['pass'])) {
$_SESSION['username'] = $_POST['name'];
echo "Erfolgreich eingeloggt!";
}
else {
echo "Ihr Passwort ist falsch!";
}
}

Alles anzeigen

SeBa · 20. August 2007, 18:49

Da ich annehme, dass das Passwort bei dir in der Datenbank schon als MD5-Hash steht muss das md5() in Zeile 16 weg.

CookIe · 20. August 2007, 18:49

Mach die Abfrage doch so, wie ich es im zweiten Beitrag editiert hatte.

KageMurai · 20. August 2007, 18:52

So, jetzt noch mal (nachdem ich alles geändert habe) den kompletten Code, der mir immer noch das gleiche liefert:

Quellcode

<?
include ("configs.php");
if (!isset($_POST['name']) || !isset($_POST['pw'])) {
echo '<form action="login2.php" method="POST">';
echo 'Name: <input type="text" name="name"/><br/>';
echo 'Passwort: <input type="text" name="pw"/><br/>';
echo '<input type="submit" value="OK">';
echo '</form>';
}
else {
if ($_POST['name'] == "") {
echo "Bitte geben Sie einen Namen ein!";
}
elseif ($_POST['pw'] == "") {
echo "Bitte geben Sie ein Passwort ein!";
}
else {
$connection=mysql_connect($server,$user,$pass) or die (mysql_error());
mysql_select_db("nedias",$connection) or die (mysql_error());
$user_pass = md5($_POST['pass']);
$query = mysql_query("SELECT pass FROM user WHERE nick = '".$_POST['username']."'") or die('Auswählen fehlgeschlagen!');
$ergebnis=mysql_fetch_array($query);
if(md5($_POST['pw']) == $ergebnis['pass']) {
$_SESSION['username'] = $_POST['name'];
echo "Erfolgreich eingeloggt!";
}
else {
echo "Ihr Passwort ist falsch!";
}
}
}
?>

Alles anzeigen

CookIe · 20. August 2007, 18:54

Blöde Frage, aber wie hast Du das Passwort in der Datenbank gespeichert?
MD5, sicher? Mach doch mal bitte ein Bild aus deinem PHP My Admin, von der Tabelle.

KageMurai · 20. August 2007, 18:58

Hallo,

hier das Bild:

5load.de/img_21859_gzg.gif

KageMurai

CookIe · 20. August 2007, 19:04

Ich meinte eins, wo man das mit dem PWD auch sieht, aber egal.

Du weißt bestimmt, wie das Passwort heißt, richtig?
Dann überprüf' doch direkt, ob "meinpasswort" mit $ergebnis[pwd] übereinstimmt?

Schau dann was so raus kommt? ..

KageMurai · 20. August 2007, 19:35

Ja, beide Passwörter sind genau gleich. Mit dem alten Skript hat es ja funktioniert. Aber was kann dann falsch sein?

kageMurai

Torben Brodt · 20. August 2007, 20:37

#24 Die POST Variable ist name, nicht username

KageMurai · 21. August 2007, 09:29

Daran lag es - Danke!

BennyBunny · 21. August 2007, 12:38

Ein kleiner Tipp am Rande:

Wenn du sowieso auf den assotiativen Teil des Array zurückgreifst, den mysql_fetch_array zurück gibt, dann kannst du auch gleich mysql_fetch_assoc benutzen. Wenn du jetzt nicht genau wissen solltest, was ich meine. Hier ein kleines Beispiel:

Quellcode

<?php
mysql_connect("localhost", "root", "");
mysql_select_db("test");
$res_array = mysql_query("SELECT * FROM test_tabelle");
$res_assoc = mysql_query("SELECT * FROM test_tabelle");
$assoc = mysql_fetch_array($res_array);
$array = mysql_fetch_assoc($res_assoc);
print_r($assoc);
print_r($array);
?>

Alles anzeigen

Als Ergebnis bekommst du dann:

Quellcode

Array
(
[0] => 1
[feld1] => 1
[1] => 2
[feld2] => 2
[2] => 3
[feld3] => 3
)
Array
(
[feld1] => 1
[feld2] => 2
[feld3] => 3
)

Alles anzeigen

Wie du siehst verbraucht die fetch_array-Version fast das doppelte an Speicher!

"Ihr Passwort ist falsch" - Login defekt

"Ihr Passwort ist falsch" - Login defekt

Quellcode

Quellcode

Quellcode

Quellcode

Quellcode

Quellcode

Quellcode

Quellcode

Teilen

Benutzer online 1

Tags