Dla potomnych

$login = $_POST["login"];
$haslo = $_POST["haslo"];
$adres = $_SESSION["adres"];
if (isset($login) && $login=="") { $blad++; $komunikat .= "Login nie został podany!<br />";
}
$wynik=mysql_query("SELECT * FROM logowanie WHERE login='$login'"); $komunikat .= "Login <b>$login</b> nie widnieje w naszej bazie! <br />";
}
}
if (isset($haslo) && $haslo=="") { $blad++; $komunikat .= "Hasło nie zostało podane!<br />";
}
$haslo_spr = md5($haslo); $wynik=mysql_query("SELECT * FROM logowanie WHERE login='$login' AND haslo='$haslo_spr'"); $komunikat .= "Hasło nie jest prawidłowe! <br />";
}
}
if ($blad>0) {
echo "<div class='error'>".$komunikat."</div><br/>"; }
if ($_GET["login"]=="koniec") { // wylogowanie
} else if ($login<>"" and $haslo<>"") {
login='$login' and haslo='$haslo' and status>'1'");
$_SESSION["zalogowany"]="tak";
$_SESSION["login"]=$dane["login"];
$_SESSION["imie"]=$dane["imie"];
$_SESSION["status"]=$dane["status"];
}
}
Cytat
dodatkowo MD5 nie jest bezpiecznym sposobem przechowywania haseł, należy stosować rozwiązania z solą, np. phpass który jest obecnie powszechnie używany przez phpbb3, wordpress itd.
kod nie jest także zabezpieczony przed atakami SQL INJECTION
Słuszna uwaga, muszę nad tym posiedzieć