Skrypt logowania - prośba o sprawdzenie Opcje

[tara]

Witam,

czy taki skrypt jest bezpieczny ? Czy można go łatwo złamać przez wykrozystanie jakiegoś myku ?

login.php

[PHP] pobierz, plaintext 
<?
session_start(); 
session_name(admin_login); 
function logowanie() 
{ 
  echo(&#092;"<form method=\"POST\">n 
   <h1> Zaloguj sie! </h1><br>n 
   User: <input type=&#092;"text\" name=\"login\"><br>n 
   Pass: <input type=&#092;"password\" name=\"haslo\"><br>n 
   <input type=&#092;"submit\" name=\"submit\" value=\"Zaloguj\"><br>n 
   <br> 
  </form>n&#092;"); 
} 
 
 
 
if(!isset($submit)) 
{ 
   logowanie(); 
} 
else 
{ 
   $db=mysql_connect('') or die(&#092;"Nie można się połšczyć z serwerem\"); 
	   mysql_select_db('') or die(&#092;"Problem z łšczeniem się z bazš danych :(\");
  $sql=&#092;"SELECT * FROM administratorzy WHERE admin = '$login'\";  
  $result=mysql_query($sql); 
  while($row = mysql_fetch_array($result)) 
  { 
   $user_pass=$row[&#092;"user_pass\"]; 
   $user_name=$row[&#092;"user_name\"];  $zmienna_używana_w_skrypcie=$komórka[\"nazwa_komórki_ze_sql\"]; 
   $ban=$row[&#092;"ban\"]; 
   if (($haslo != md5($user_pass)) || ($login != $user_name)) # jeżeli hasło / login się nie zgadzajš 
	   { 
		 echo(&#092;"Zly login/pass\"); # pokazuje \"Zly login/pass\" oraz logowanie (z wczesniej zadeklarowanej funkcji:) 
		 logowanie(); 
		 } else { # w innym przypadku (tj. login i pass sš prawdziwe) 
		session_register(&#092;"login\"); # rejestrowanie w sesji \"loginu\", \"ban\" (czy gostek ma bana czy nie (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) \" 
		session_register(&#092;"ban\"); 
		if ($ban == 1) # jeżeli user ma bana to .... 
{ 
  echo(&#092;"Masz bana. <br> <pre> Error 403 <br> Brak dostępu do zasobów </pre>\"); 
} else { # jeżeli nie ma bana to pokazuje mu się index.php 
header(&#092;"Location: zarzadzanie_admin.php?\" . SID); # SID = Session ID (takie cosik w adresie (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)  
echo(&#092;"Jeżeli Ci się nie wyœwietliła strona kliknij <a href=\"zarzadzanie_admin.php\">tutaj</a>\"); 
} 
		exit(); # koniec :) 
			 } 
		 } 
	} 
?>
[PHP] pobierz, plaintext 

header innych plików:

[PHP] pobierz, plaintext 
<? 
session_start(); 
session_name(admin_login); 
if (!isset($_SESSION[&#092;"login\"]))
{ 
header(&#092;"Location: login.php\"); 
exit(); 
if ($_SESSION[&#092;"ban\"] == 1) 
	{ 
	 echo(&#092;"Masz bana. <br> <pre> Error 403 <br> Brak dostępu do zasobów </pre>\"); 
	exit(); 
	} }
?>
[PHP] pobierz, plaintext 

Co o tyym sądzicie ? Od razu zaznaczam że skrypt ten nie jest mój.