[PHP]Logowanie + sesje Opcje

[iksior]

Witam zaczynam zabawę z sesjami i zastanawiam się co jest nie tak w poniższym kodzie?

[PHP] pobierz, plaintext 
<?php
session_start();
	if($zalogowany > "0"){
	$_SESSION['zalogowany'] = 1;
	}
	if(isset($_SESSION['zalogowany'])){
		$_SESSION['user'] = $user;
		$_SESSION['password'] = $pass;
	}
	if(!isset($_SESSION['zalogowany'])){
		$user = $_POST['user'];
		$pass = $_POST['password'];
		$con = mysql_connect("sql112.byetcluster.com", "usr", "pass") or (mysql_error());
		$zalogowany = "0";
			if(!$con)
			die(mysql_error());
		mysql_select_db("b14_13xxxxx4_db", $con) or die (mysql_error);
		$check = mysql_query('SELECT * FROM `konta` WHERE `user` = "'.$user.'" AND `password` = "'.md5($pass).'"');
		$numrows = mysql_num_rows($check);
			if($numrows == 0){
				die("Użytkownik nie istnieje \n");
			}else{
				$pass = md5($pass);
			while($row = mysql_fetch_assoc($check)){
				if($pass == $row['password']){
					$zalogowany++;
					$logedIn=mysql_query( "UPDATE konta SET zalogowany='$zalogowany' WHERE user= '$user' LIMIT 1");
					//DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE 
					die("ZALOGOWANO");
					//DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE DIE
				}
				else{
					//DIE DIE DIE DIE DIE DIE DIE DIE DIE
					die("Hasło jest niepoprawne \n");
					//DIE DIE DIE DIE DIE DIE DIE DIE DIE
				}
			}
			}
	}
?>
[PHP] pobierz, plaintext 

[Michael2318]

To może Ty nam powiedz co Tobie nie działa, a my spróbujemy wtedy znaleźć błąd...?
Pierwsze co rzuca się w oczy:

[PHP] pobierz, plaintext 
if($zalogowany > "0"){
[PHP] pobierz, plaintext 

Odwołujesz się do zmiennej $zalogowany, a jej nigdzie wcześniej nie zdefiniowałeś, zmienna ta zawsze będzie równa 0 lub FALSE.

[PHP] pobierz, plaintext 
	if(isset($_SESSION['zalogowany'])){
$_SESSION['user'] = $user;
$_SESSION['password'] = $pass;
}
[PHP] pobierz, plaintext 

Tu też przypisujesz do zmiennych sesyjnych zera bo te zmienne w Twoim kodzie nie istnieją, chyba że coś ukryłeś ;>

[PHP] pobierz, plaintext 
$check = mysql_query('SELECT * FROM `konta` WHERE `user` = "'.$user.'" AND `password` = "'.md5($pass).'"');
[PHP] pobierz, plaintext 

zmienną $user należy filtrować, aby zabezpieczyć się przez sqli.

Ten post edytował Michael2318 25.05.2013, 08:56:52

[iksior]

$zalogowany="0"; w 14 linijce a w 26 $zalogowany++
czyli tutaj zle?

[Michael2318]

Najpierw się przypisuje zmienne, potem się do nich odwołuje. Nie ta kolejność.