Passwortabfrage

[kave78]

hallo

habe auf meiner Seite eine abfrage bei der der user einen benutzernamen und ein passwort eingebn soll. abgefragt wird bei mysql db.

jetzt ist es aber so, wenn nichts eingegeben wird oder ein falscher benutzername ohne passwort kommt der user trotzdem hinnein.
könnt ihr da helfen?

$anfrage="SELECT passwd FROM Tabelle WHERE kenn = '".$_GET['benutzername']."'";
$ergebnis=mysql_query($anfrage);
$daten = mysql_fetch_object($ergebnis);
if ( $daten->[0]  != $_GET['passwort'] ) die("Pech gehabt"); 

[davidovich]

Schau dir mal folgendes an (wenn dir was unklar ist, einfach fragen):

<?php
$benutzername = (isset($_GET["benutzername"]) && strlen($_GET["benutzername"]) > 0) ? 
                mysql_escape_string($_GET["benutzername"]) : "";
$passwort = (isset($_GET["passwort"]) && strlen($_GET["passwort"]) > 0) ? 
            mysql_escape_string($_GET["passwort"]) : "";

if($benutzername == "" || $passwort == "")  //Fehlermeldung
else {
    $query = "SELECT userID FROM tabelle WHERE benutzername = '$benutzername' 
             AND passwort = '$passwort'";
    $ergebnis=mysql_query($query) or die(mysql_error());    
    $num = mysql_num_rows($ergebnis);
    if($num > 0 ) //Benutzer einloggen
    else //Fehlermeldung
}
?>

[penizillin]

bitte lesen und befolgen!
http://www.php-resource.de/forum/sho...threadid=47906

(das grundrecht auf gescrollefreies bild darf nicht verletzt werden)

[davidovich]

Ich kann ja au nichts dafür wenn meine Abfrage so lange ist naja werde mal ein paar [enter] reinhauen ^^

So besser?

[penizillin]

viel besser!
noch die sql-query kleinhacken?...
komm, ein zeilenumbruch ist noch drin..

[davidovich]

Also bei mir passts so rein, aber wennn du magst

[ghostgambler]

<?php
$benutzername = (isset($_GET["benutzername"]) && strlen($_GET["benutzername"]) > 0) ? 
                mysql_escape_string($_GET["benutzername"]) : "";
$passwort = (isset($_GET["passwort"]) && strlen($_GET["passwort"]) > 0) ? 
            mysql_escape_string($_GET["passwort"]) : "";

if($benutzername == "" || $passwort == "")  //Fehlermeldung
else {
    $query = "SELECT userID FROM tabelle WHERE benutzername LIKE '$benutzername' 
             AND passwort = '$passwort'";
    $ergebnis=mysql_query($query) or die(mysql_error());    
    $num = mysql_num_rows($ergebnis);
    if($num > 0 ) //Benutzer einloggen
    else //Fehlermeldung
}
?>

vielleicht besser so? Dann wird groß/kleinschreibung ignoriert ^^

[davidovich]

naja geschmacksache, ibei mir müssen die User halt ihren Namen richtig schreiben können

[Hopka]

$daten->[0] 

Was soll das denn bitte sein?

Entweder du willst Arrays (mysql_fetch_array, mysql_fetch_assoc) oder Objekte (mysql_fetch_object) von der DB kriegen - dann musst du sie aber hinterher auch auf die Variablen dem entsprechend zugreifen, und nicht beides irgendwie vermischen.

[kave78]

ja vielen dank ...eure beiträge haben mir sehr weitergeholfen


ps hopka: das mysql_fetch_object war vorher ein mysql_fetch_row
deswegen war das daten[0] noch drinne ;-)