Logowanie Opcje

[Hectic]

Czy może mi ktoś powiedzieć czy jest to poprawny kod funkcji logowania?

[PHP] pobierz, plaintext 
<?
$login=$_POST['login'];
$password = md5($_POST['password']);
 
$result = mysql_query ("SELECT password FROM `admin` WHERE login = '$login' "); // pobranie hasła do podanego loginu
$pas = mysql_fetch_array($result);
 
 
echo'
 
<form method="post">
<table border="0" cellspacing="0" cellpadding="0">
<tr>
<td height="5"></TD>
</tr>
<tr>
<td>Login: </td>
<td width="5"></td>
<td colspan="2"><input type="Text" name="login"></td>
</tr>
<tr>
<td height="5"></TD>
</tr>
<tr>
<td>Hasło: </td>
<td width="5"></td>
<td><input type="password" name="password"></td>
</tr>
<tr>
<td height="5"></TD>
</tr>
<tr>
<td colspan="3" align="center"><input type="Submit" name="submit" value="Wyślij"></td>
</tr>
</table></form>';
 
if ($password == $pas['password'])
 
echo'Hasło ok';
 
else 
 
echo 'Złe hasło';
?>
[PHP] pobierz, plaintext 

////Edit

hasło jest kodowane w MD5 (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Ten post edytował Hectic 1.03.2006, 15:03:45

[nospor]

@qbejs

Masz tu mój sprawdzony system

Kto go sprawdzil? na czym się sprawdzil? Niedawno dales go do oceny. śmiem podejrzewac ze nie jest on jednak sprawdzony.

[PHP] pobierz, plaintext 
<?php
$login=htmlspecialchars(trim($_POST['login']));
?>
[PHP] pobierz, plaintext 

Widze że nie zapoznales się z topiciem, do ktorego podalem linka w mym pierwszym poście.

[PHP] pobierz, plaintext 
<?php
$pass=md5(md5($_POST['pass']));
?>
[PHP] pobierz, plaintext 

Juz chyba o tym rozmawialismy:
http://forum.php.pl/index.php?showtopic=42615&hl=
Przestań to rozpowszechniac, bo naprawdę ktos uwierzy ze to prawda.