Am urmatorul formular de autentificare:

<form action="procesare.php" method="post">

username:<input type="text" name="username"  maxlength="222" />

password:<input type="password" name="password" maxlength="222" />

<input type="submit" name="baga_mare" value="Baga mare!"  />

</form>

si pagina procesare.php care imi prelucreaza datele din formular:

<?php if($_POST['username'] == 'username`ul_meu' && $_POST['password'] == 'parola_mea'){

echo'<table>...</table>//un tabel oarecare
';}
else
{echo 'Nu esti autorizat sa accesezi aceasta pagina web!!';}
?>

Vreau sa stiu daca mai trebuie sa adaug ceva la codul PHP ca sa nu am vulnerabilitati de securitate.Vreau sa va uitati la cod si sa imi spuneti ce buguri are.

Multumesc.

This post was last modified: 08-02-2008 03:20 AM by dtbd1916.

E prea mic ca sa se poata exploata ceva

O singura problema observ si anume modul in care faci comparatia. == se foloseste doar pentru numere. Daca ai string-uri iti recomand strcmp() sau operatorul ===. Daca folosesti ==, "1" == "1e0" iti va returna true.

Merci.Deci in loc de

if($_POST['username'] == 'username`ul_meu' && $_POST['password'] =='parola_mea')

trebuie sa scriu

if($_POST['username'] === 'username`ul_meu' && $_POST['password'] === 'parola_mea')?

Exact

Sau, varianta mai eleganta: