[PHP]Logowanie nie działa Opcje

[porywacz]

Hejka, napisałem skrypt logowania ale nie chce mi działać. Loguje mnie niezależnie od tego czy wpisuje poprawne dane do pola loginu i hasła ;(
Proszę o pomoc (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php session_start();
 
$login = $_POST['login'];
$haslo = $_POST['password'];
$login = stripslashes($login);
$haslo = stripslashes($haslo);
 
include('connect_mysql.php');
 
$login = mysql_real_escape_string($login);
$haslo = mysql_real_escape_string($haslo);
 
if(isset($login)) {
	if(empty($login) or empty($haslo)) {
		echo('Logowanie nie powiodło się musisz wypełić szystkie pola!');
	} elseif(!empty($login) && !empty($haslo)) {
		$haslo_md5 = md5($haslo);
 
		$users_login = mysql_query("select count(*) from `uzytkownicy` where `login` = '".$login."'");	
		$users_haslo = mysql_query("select count(*) from `uzytkownicy` where `haslo` = '".$haslo_md5."'");
 
		$count_login = mysql_num_rows($users_login);
		$count_haslo = mysql_num_rows($users_haslo);
 
		if($count_login = 1 && $count_haslo = 1) {
			$_SESSION['login'] = $login;
 
			echo('Pomyślnie zalogowałeś się jako <font color="red">	'.$_SESSION['login'].'</font>.');
		} else {
			echo('Logowanie się nie powiodło, sprawdź czy poprawnie wypełniłeś wszystkie pola.');
		}
	}
}
?>
[PHP] pobierz, plaintext 

[klocu]

Metoda jest delikatnie mówiąc... bez sensu.
Bo sprawdzając w bazie ilość hash'y md5 podanego hasła możesz przypadkiem otrzymać wartość większa niż 1.
Raczej lepiej jest to załatwić jednym zapytaniem i przebudować całość.

a to:

[PHP] pobierz, plaintext 
if($count_login = 1 && $count_haslo = 1)
[PHP] pobierz, plaintext 

nie jest poprawnym zapisaniem warunku, który jak sądzę chcesz uzyskać.

dalej:

[PHP] pobierz, plaintext 
if(isset($login))
[PHP] pobierz, plaintext 

będzie spełnione bo sam definiujesz daną zmienną, a że będzie mieć wartość NULL. ale istnieć będzie.

Ten post edytował klocu 16.04.2012, 20:13:03