[PHP] Skrypt Logowania [Zabezpieczenia] Opcje

[Reptile ReX]

Witam chciałbym zapytać czy mój poniższy kod na logowanie jest bezpieczny, jeżeli nie, prosiłbym o pomoc w zabezpieczeniu go.
Podejrzewam luki w sesjach, ale proszę was o pomoc.

PS: Oczywiście katalog z plikami .txt jest zabezpieczony przez serwer + .htaccess

[PHP] pobierz, plaintext 
<?php 
    if (isset($_SESSION['logon'])) {
        $NazwaPliku = "xxx\cxx.txt";   
        $Otworz = fopen($NazwaPliku,"r");
        $Odczyt = fread($Otworz,filesize($NazwaPliku));
        echo("<br /><center><form action='downloads/xxx.exe'><input type='submit' value='--->>> Pobierz Serwer <<<---' class='submit' /></form></center>");
	    echo("<br /><center><form action='delete.php'><input type='submit' value='--->>> Skasuj Logi <<<---' class='submit' /></form></center>");
	    echo("<br /><center><a href='?logout=yes'>--->>> Wyloguj Się <<<---</a></center>");
        echo("<br /><br />".nl2br($Odczyt)."<br />");
 
        if ($_GET['logout'] == 'yes') {
 
            session_destroy();
            header('Location: index.php');
            }
        } else {
 
            if (isset($_POST['logme'])) {
 
    if (strip_tags(htmlspecialchars($_POST["nick"], ENT_QUOTES)) == "ReptileReX" and strip_tags(htmlspecialchars($_POST["X09261x"], ENT_QUOTES)) == "abc123") {
 
       $_SESSION['logon'] = strip_tags(htmlspecialchars($_POST["nick"], ENT_QUOTES));
       header('Location: index.php');
 
    } else {
 
        Echo("<center>Podane dane są nieprawidłowe.</center>");
    }
 
            } else {
 
                echo ("<center><form action='' method='post'>");
                echo ("Nazwa: <input type=\"text\" class=\"input\" name=\"nick\" size=\"25\" /><br />");
                echo ("Hasło:   <input type=\"password\" name=\"X09261x\" class=\"input\" size=\"25\" /><br /><br />");
                echo ("<input type=\"submit\" value=\"Zaloguj Mnie !\" name=\"logme\" class=\"submit\" />");
                echo ("</form></center>");
 
            }
        }
 
?>
[PHP] pobierz, plaintext 

Ten post edytował Reptile ReX 25.09.2010, 13:29:12

[Fifi209]

Po co te cuda?

[PHP] pobierz, plaintext 
 if (strip_tags(htmlspecialchars($_POST["nick"], ENT_QUOTES)) == "ReptileReX" and strip_tags(htmlspecialchars($_POST["X09261x"], ENT_QUOTES)) == "abc123") {
 
 
[PHP] pobierz, plaintext 

Zrób sobie tak:

[PHP] pobierz, plaintext 
 
if (md5($_POST['nick']) == md5('ReptileReX') && md5($_POST['X09261x']) == md5('abc123')) {
[PHP] pobierz, plaintext 

[Reptile ReX]

ok, dzięki, coś więcej ?

[Atomic_Cookie`]

Nie chce mi się dokładnie czytać, ale przydałoby się session fixating.
tzn:

[PHP] pobierz, plaintext 
 
 if (isset($_SESSION['logon'])) {
//Sprawdzamy czy ip zapisane przy logowaniu się jest takie same jak teraz
if($_SESSION['ip'] != $_SERVER['REMOTE_ADDR']) session_regenerate_id();
}
//Jakiś kod
if (isset($_POST['logme'])) {
 
    if (strip_tags(htmlspecialchars($_POST["nick"], ENT_QUOTES)) == "ReptileReX" and strip_tags(htmlspecialchars($_POST["X09261x"], ENT_QUOTES)) == "abc123") {
 $_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
       $_SESSION['logon'] = strip_tags(htmlspecialchars($_POST["nick"], ENT_QUOTES));
       header('Location: index.php');
 
    }
[PHP] pobierz, plaintext 

[Reptile ReX]

Ok dziękuje, czekam na kolejne propozycje