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 ja mam tylko pytanie po co to rzutowanie na integer przy id użytkownika? Przecież ta wartość jest pobierana z bazy i trzymana w sesji, a więc wiemy, że jest poprawna.

no może akurat w tym przypadku w kodzie proceduralnym nie będzie to konieczne, ale przecież w praktyce zmienne są podawane do metody jako argumenty i zapytanie trzeba odpowiednio zabezpieczyć bez wnikania skąd to będzie pochodziło bo różnie może być metoda użyta. Wydaje mi się, że też jako początkujący lepiej jak się nauczy zabezpieczać 100% przypadków, bo potem są takie sytuacje, gdzie bez namysłu skopiuje sobie gotowe zapytanie na inną podstronę zamieni $_SESSION na $_GET i luka gotowa

A w zapytaniu tak samo na zadziała '1'(string) oraz 1(integer).

Tak, w mysql tak samo zadziała

[PHP] pobierz, plaintext 
user_id = 1 oraz user_id = '1'
[PHP] pobierz, plaintext 

więc, tutaj można mówić jedynie o dobrym nawyku programowania, skoro mamy pole typu int to powinniśmy podawać int, a nie parsować string