[PHP]Czy to logowanie jest bezpieczne? Opcje

[topik53]

Witam,
napisałem taki skrypt logowania, czy jest on bezpieczny, czy da sie jakoś ominąć.
Ja tutaj nic nie widze, ale wole zapytać bardziej doświadczonych koderów.

[PHP] pobierz, plaintext 
function _login($login,$pass){
    $select_pass = "SELECT password FROM cn_users WHERE login='$login'";
    $sql_select_pass = mysql_query($select_pass);
    $result_select_pass = mysql_fetch_assoc($sql_select_pass);
    $password = $result_select_pass['password'];
    $pass = sha1(md5($pass));
    if($pass == $password)
    {
      //set_session;
    }
[PHP] pobierz, plaintext 

Ten post edytował topik53 18.10.2011, 14:33:06

[croc]

Funkcja jest podatna na SQL Injection, bi nie filtrujesz zmiennej login. Poza tym możesz to zrobić zapytaniem, ktore sprawdzi od razu zgodność loginu i hasła, bez pobierania hasła do zmiennej.