Login Skript mit Fehlversuchen

[TBW]

Hallo,

ich versuche eine Loginskript zu erstellen dass bei 3 falschen eingaben von Passwort das Login für den Benutzer sperren soll.
Irgendetwas läuft dabei schief, mit Hilfe der Bücher bekomme ich es nicht.
Bitte um Rat von denen die sich damit auskennen.
Vielen Dank.

PHP-Code:

$password = $_POST['pass'];
$user = $_POST['name'];

$fehler = $user_daten['fehler'];
$fehler_neu = $fehler + 1;

$user = mysql_query("select * from login where Name='$user' and fehler < 3");
$user_daten = mysql_fetch_array($user);

while($row = mysql_fetch_array($user_daten)){
  $checkuser = $row['Name'];
  $checkpassword = $row['Passwort'];}

  if($user == $checkuser && $password ==  $checkpassword) {

    echo "Du wurdest erfolgreich eingeloggt !!";

    } else {

      mysql_query("update login set fehler='$fehler_neu' where Name='$user'");

    echo "Falsches Passwort";
  } 


[Master_T2]

Erstmal solltest du in der Tabelle eine neue Spalte anlegen mit einer User_id die fortlaufend durchnummerriert damit du jeden User eindutig zuordnen kannst.

Zudem hast du da irgendwie ein doppelgemoppeltes mysql_fetch_array drin !?!

PHP-Code:

$user = mysql_query("select * from login where Name='$user' and fehler < 3"); 
$user_daten = mysql_fetch_array($user); 

while($row = mysql_fetch_array($user_daten)){ 


Hier kann ja wohl was nicht stimmen

[TBW]

Hallo,

vielen Dank..

Ich kenne mich mit PHP überhaupt nicht aus, ich habe mir ein Buch aus der Stadtbücherei ausgeliehen versuchee damit das hinzubekommen.
Ich habe folgende Spalten
Name varchar(20) Nein
Passwort varchar(20) Nein
fehler tinyint(4) Nein 0
ID

Doppeltes mysql_fetch_array ??

auch wenn ich die Zeile

$user_daten = mysql_fetch_array($user);

rausnehme, tut sich gar nichts !!

[H3ck|0]

PHP-Code:

$password = $_POST['pass']; 
$user = $_POST['name']; 

//$fehler = $user_daten['fehler']; $user_daten ist hier noch leer..
//$fehler_neu = $fehler + 1; in der alten Version wäre $fehler_neu immer 1...

$user = mysql_query("select * from `login` where `Name`='$user' and `fehler` < 3 limit 0,1;"); 
 //limit sorgt dafür, dass nur ein Datensatz ausgelesen wird..
 
$user_daten = mysql_fetch_array($user); 

//while($row = mysql_fetch_array($user_daten)){ 
// unnötig da nur 1 Datensatz aus der DB gezogen wird..
  $checkuser = $user_daten['Name']; 
  $checkpassword = $user_daten['Passwort'];
  $fehler = $user_daten['fehler'];  
// hier ist auch die Anzahl der Fehlversuche des users bekannt..
//}
  if($user == $checkuser && $password ==  $checkpassword) { 

    echo "Du wurdest erfolgreich eingeloggt !!"; 

    } else { 
      $fehler_neu = $fehler + 1; // der Fehlversuch wird ja erst hier bemerkt..
      mysql_query("update `login` set `fehler`='$fehler_neu' where `Name`='$user'"); 

    echo "Falsches Passwort"; 
  } 


So sollte es funktionieren...

Die ID Spalte der Tabelle sollte die Eigenschaften int(3) NOT NULL auto_increment haben und als PRIMARY KEY definiert sein...

[TBW]

Hallo,

vielen Dank..
ich bekomme die Meldung "Falsches Passwort" auch wenn ich den richtigen eingebe.
Die Fehlversuche werden auch nicht in dei Spalte "fehler" eingetragen.

[H3ck|0]

Setz bitte mal ein "echo mysql_error($dein_verbindungs_handle);" hinter die beiden
Datenbankabfragen...

Zusätzlich möchte ich dir mal Mikes Login Tutorial an die Hand legen
http://www.traum-projekt.com/forum/s...ad.php?t=35220
Dort werden unter anderem auch einige Sicherheitsaspekte berücksichtigt die bei
bei deinem Script vernachlässigt werden..