[skrypt] Logowanie Opcje

[sebekzosw]

Witam!

Mam taki skrypt logowania opartego o sesje - moje pytanie brzmi: Czy taki skrypt jest bezpieczny?:

[PHP] pobierz, plaintext 
<?php
    function DodawanieDoBazy($ciag) {
        $wynik = stripslashes(strip_tags(addslashes(trim($ciag))));
        $wynik = str_replace(array("\"", "'", "\\", '\"', "\'", "<", ">", "&nbsp;"), array("&quot;", "'", "\", "&quot;", "'", "&lt;", "&gt;", " "), $wynik);
        $wynik = preg_replace(array("/\=/","/\#/","/\sOR\s/"), "", $wynik);
        return $wynik;
    }
 
 
    if(isset($_POST["logowanie"]) && isset($_POST['login']) && isset($_POST['login'])) {
        $logon_failure = '';
        if(empty($_POST["login"])) { $logon_failure .= "Wpisz login<br />"; }
        if(empty($_POST["password"])) { $logon_failure .= "Wpisz hasło<br />"; }
        $login = DodawanieDoBazy($_POST["login"]);
        $password = md5($_POST["password"]);
 
        if(empty($logon_failure)) {
            if(!mysql_fetch_row(mysql_query("SELECT * FROM users WHERE login='".$login."' AND pass='".$password."'"))) {
                $logon_failure .= "Nieprawidłowy login lub hasło";
            } elseif(!mysql_fetch_row(mysql_query("SELECT * FROM users WHERE login='".$login."' AND activity='1'"))) {
                $logon_failure .= "Konto jest nieaktywne";
            }
 
            if(empty($logon_failure)) {
                $_SESSION["login"] = $login;
                 $_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
                 $_SESSION['browser'] = $_SERVER['HTTP_USER_AGENT'];
            } else {
                echo $logon_failure;
            }
        } else {
            echo $logon_failure;
        }
    }
    echo "<div id=\"menu\">";
    if(UZYTKOWNIK) {
        echo "Zalogowany jako ".$_SESSION["login"];
    } else {
 
 
?>
    <form action="" method="post" id="logowanie">
        <fieldset>
            <dl>
                <dt>Login</dt>
                 <dd><input type="text" name="login" id="login" value="Login" onblur="if(this.value=='') this.value='Login';" onfocus="if(this.value=='Login') this.value='';" /></dd>
 
                <dt>Hasło:</dt>
                 <dd><input type="password" name="password" id="password" value="Hasło" onblur="if(this.value=='') this.value='Hasło';" onfocus="if(this.value=='Hasło') this.value='';" /></dd>
            </dl>
        </fieldset>
        <div style="float:right;"><input type="submit" name="logowanie" value="Zaloguj się" onclick="WyslijFormularz('logowanie'); return false;" /></div>
    </form>
    <img src="http://test.php.pl/www/gfx/login_loading.gif" id="ladowanie" alt="" style="display:none;float:left;" />
    <div id="wynik"></div>
 
    <p><a href="/rejestracja/">Rejestracja</a>
        <br />
        <a href="/przypomnij/">Przypomnij hasło</a>
 
 
 <?
 }
 ?>
 </div>
[PHP] pobierz, plaintext 

Wszystkie sugestie na temat polepszenia mile widziane (IMG:style_emoticons/default/smile.gif)

Ten post edytował erix 19.08.2009, 23:10:18

Powód edycji: [erix] przeniosłem

[l0ud]

Tak, o 7 minut. Co do htmlspecialchars, robiła to poprzednia wersja funkcji, a na nią patrzałem (mój bład). Niemniej w takiej formie jak teraz tym bardziej należałoby o tym wspomnieć, bo wyrzucenie zmiennej bez filtracji to dziura xss.
btw. właśnie wywołałeś bezsensowną dyskusję na 5 postów (IMG:style_emoticons/default/tongue.gif) Dużo lepiej by było, jakbyś w takich sprawach po prostu napisał na PW.

[Fifi209]

Dyskusja jest też na temat. (IMG:style_emoticons/default/winksmiley.jpg)

Napisałeś, przy wyrzucaniu z bazy używać htmlspecialchars tylko powiedz mi jaki ma sens ciągłe obciążanie parsera jak można to zrobić raz przy zapisie do bazy?