Logowanie, pomoc przy logowaniu Opcje

[seba199696]

[PHP] pobierz, plaintext 
<?php
session_start();
include 'dbc.php'; //polaczenie z baza danych
	$user_login = htmlspecialchars(stripslashes(strip_tags(trim($_POST['username']))));
 
 
$pass = md5(htmlspecialchars(stripslashes(strip_tags(trim($_POST['password'])))));
$user_password = sha1($pass.$user_login);
 
 
     $user_log = "user_login='$user_login'";
	 $user_pass = "user_password='$user_password'";
 
 $result = mysql_query("SELECT `id`,`user_login`,`user_password`,`banned` FROM users WHERE 
           $user_log AND $user_pass AND `banned` = '0'
			") or die (mysql_error()); 
 
 
	 if (mysql_num_rows($result) == 1) {
 
	   session_regenerate_id (true);
		$_SESSION['user_name'] = $user_login;
		header('Location: index2.php');
 
	 } else {
		echo "Logowanie się nie powiodło :("; 
 
	 }
?>
[PHP] pobierz, plaintext 

Witam, napisałem skrypt logujący użytkownika wszystko działa prawidłowo lecz chcę się dowiedzieć w jaki sposób zabezpieczyć taki skrypt przed atakami. Robię coś źle? Jakieś propozycję?

Dzięki

[seba199696]

Poprawiłem skrypt, teraz dobrze?

[PHP] pobierz, plaintext 
<?php
session_start();
// Connect
define ("DB_HOST", "xxx"); // set database host
define ("DB_USER", "xxx"); // set database user
define ("DB_PASS","xxx"); // set database password
define ("DB_NAME","users"); // set database name
 
$link = mysql_connect(DB_HOST, DB_USER, DB_PASS) or die("Couldn't make connection.");
$db = mysql_select_db(DB_NAME, $link) or die("Couldn't select database");
 
     if(get_magic_quotes_gpc()) {
            $uzytkownik = stripslashes($_POST['login']);
			$haslo = stripslashes(sha1($_POST['password'].'sad8%$9sdk'));
        } else {
            $uzytkownik = $_POST['login'];
            $haslo = sha1($_POST['password'].'sad8%$9sdk');
        }
$query = sprintf("SELECT * FROM users WHERE user_login='%s' AND user_haslo='%s'",
            mysql_real_escape_string($uzytkownik),
            mysql_real_escape_string($haslo));
 
		mysql_query($query, $link);
 
		   if (mysql_affected_rows($link) > 0) {   
           session_regenerate_id (true);
 
			$_SESSION['user_name'] = $uzytkownik;
                        $_SESSION['HTTP_USER_AGENT'] = sha1($_SERVER['HTTP_USER_AGENT'].'dsd#$%^');
		    header('Location: index2.php');
 
        } else {
			echo "Logowanie nie powidło się :)";
		}
?>
[PHP] pobierz, plaintext 

Ten post edytował seba199696 8.09.2011, 22:24:01