[php] logwanie poprawne Opcje

[icetab]

Czy ten skrypt na logowanie jest zabezpieczony i czy jest poprawnie napisany ?

[PHP] pobierz, plaintext 
<?php
session_start();
 if(isset($_POST['login'])){
 $login = $_POST['login'];
 }
 else{
 $login='';
 }
  if(isset($_POST['haslo'])){
 $haslo = $_POST['haslo'];
 }
 else{
 $haslo='';
 }
 $l = 'login';
 $h = 'haslo';
 
 if (($login == $l) and ($haslo == $h)){
  $_SESSION['logowanie'] = '1';
 }
 else {
if (!isset($_SESSION['logowanie'])){
 $_SESSION['logowanie'] = '0';
 }
 }
if (isset($_SESSION['logowanie']) and ($_SESSION['logowanie'] == 1)){
header ('location: strona.php');
}
else {
echo 'Blad logowania!';
}
?>
[PHP] pobierz, plaintext 

Ten post edytował icetab 20.04.2007, 21:24:48

[xian.pl]

Jeśli ma być to do zabawy typu hackme to oceniam to 3/10 punktów
w sesji lepiej przechowywać md5(hasło);

[starach]

Raczej nikt ci tutaj nie napisze gotowego skryptu. [to tylko żeby nie było zbyt oftopicznie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) ]

Jeśli ma być to do zabawy typu hackme to oceniam to 3/10 punktów
w sesji lepiej przechowywać md5(hasło);

A dlaczego w sesji hasło powinno być w md5 ?
Przecież sesja jest po stronie serwera i chyba tylko print_r($_SESSION) można podejrzeć jej zawartość.