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

[tehaha]

a to co za wynalazek?

[HTML] pobierz, plaintext 
$pass = md5(htmlspecialchars(stripslashes(strip_tags(trim($_POST['password'])))));
[HTML] pobierz, plaintext 

wepchałeś tu całą masę funkcji a i tak nie zabezpieczyłeś:

[HTML] pobierz, plaintext 
$user_login = htmlspecialchars(stripslashes(strip_tags(trim($_POST['username']))));
[HTML] pobierz, plaintext 

W taki sposób do niczego nie dojdziesz...dowiedz się co robią te funkcje i w jakich sytuacjach je używać i rób to z głową, a nie na zasadzie "wepcham wszystko to będzie ok", do zabezpieczania zapytań używa się mysql_real_escape_string() dla string oraz np. rzutowanie na int (int) dla int, poczytaj sobie o sql injection, bo nawet ta funkcja w pewnych sytuacjach może nie być wystarczająca