Форумы портала PHP.SU :: Версия для печати :: Авторизация. Как вам код?

1. Serpanok - 17 Июня, 2011 - 09:51:21 - перейти к сообщению

Всем привет!

Посмотрите код, и скажите какие тут есть щели для взломщика и как их можно латануть?

<?
function screening($text) {
        return trim(iconv("utf-8", "Windows-1251", htmlspecialchars(addslashes(stripslashes($text)))));
}
 if($_GET["mod"] == "login" and isset($_POST["login"]) and isset($_POST["pass"]) and !isset($_SESSION["id"]))
{
        $login = screening($_POST["login"]);
        $pass = screening($_POST["pass"]);      
        if($login == "" or $login == " " or $pass == "" or $pass == " ")
        {
                $_SESSION["see"] = "Проверьте правильность заполнения формы!";
                exit(header("Location: /"));
        }
        $pass = md5($pass);
        
        $result_login = mysql_query("SELECT * FROM users WHERE login='$login' and pass='$pass'",$db);
        $rows_login = mysql_num_rows($result_login); 
        if($rows_login == 1)
        {
                $myrow_login = mysql_fetch_array($result_login);
                $_SESSION["id"] = $myrow_login["id"];
                $_SESSION["priv"]       = array(
                        'add'                   => $myrow_login["add"],
                        'edit'                  => $myrow_login["edit"],
                        'add_ostatok'   => $myrow_login["add_ostatok"],
                        'vrabote'               => $myrow_login["vrabote"],
                        'vsklade'               => $myrow_login["vsklade"],
                        'rept'                  => $myrow_login["rept"],
                        'archive'               => $myrow_login["archive"],
                        'order_adds'    => $myrow_login["order_adds"],
                        'settings'              => $myrow_login["settings"],
                        'graph'                 => $myrow_login["graph"],
                        'graph_add'             => $myrow_login["graph_add"],
                        'print'                 => $myrow_login["print"],
                        'docs'                  => $myrow_login["docs"]
                );
                exit(header("Location: /"));
        }
        else
        {
                $_SESSION["see"] = "Неправельный логин или пароль!";
                exit(header("Location: /"));    
        }
} 
if(isset($_SESSION["see"])) echo $_SESSION["see"];
?>
 
<form action="/?mod=login" method="post">
Логин:<input name="login" type="text" />Пароль:<input name="pass" type="password" /><input type="submit" value="Вход" />
</form>

Обработчик и форма находятся на разных страницах, тут я их кинул на одну для вашего удобства.

Помогите сделать авторизацию на высшем уровне(просто там конфиденциальная информация).

Если нужно ещё какая инфа, скажу...

Заранее спасибо!