[Pytanie] - Logowanie MYSQL, [Pytanie] - Logowanie MYSQL Opcje

[BUKKICIK]

Witam,
mam pewien problem, ponieważ nie wiem jak ustawić pewną rzecz. Mianowicie: gdy klikam zaloguj się, a inputy są puste to przenosi dalej, a nie powinno! Pomoże ktoś(IMG:style_emoticons/default/questionmark.gif) ? Słyszałem o "!empty", ale nie wiem gdzie to dać (IMG:style_emoticons/default/haha.gif) .

[PHP] pobierz, plaintext 
<?php
session_start();
ob_start();
 
include("db_login.php");
 
$login = $_POST["login"];
$haslo = $_POST["password"];
 
$zapytanie = mysql_query("SELECT Login FROM users WHERE Login = '$login' ");
$wynik_zapytania = mysql_fetch_array($zapytanie);
 
if($wynik_zapytania[0] != $login){
	header("Location: ../index.html");
}else{
 
	$zapytanie = mysql_query("SELECT Haslo FROM users WHERE Login = '$login' ");
	$wynik_zapytania = mysql_fetch_array($zapytanie);
 
	if($haslo == $wynik_zapytania[0]){
 
		$zapytanie = mysql_query("SELECT ID, Login, EMail FROM users WHERE Login = '$login' ");
		$wynik_zapytania = mysql_fetch_array($zapytanie);
 
		$_SESSION["zalogowany"] = 1;
		$_SESSION["user_id"] = $wynik_zapytania[0];
		$_SESSION["user_login"] = $wynik_zapytania[1];
		$_SESSION["user_mail"] = $wynik_zapytania[2];
 
		mysql_query("UPDATE users SET LData = NOW() WHERE Login = '$login' ");
 
		header("Location: panel/index.php");
	}else{
		header("Location: ../index.html");
	}
}
ob_end_flush();
?>
[PHP] pobierz, plaintext 

Ten post edytował BUKKICIK 23.04.2015, 10:00:35

[viking]

Przekirowujesz wszędzie header() jeżeli występują błędy. Wypisz odpowiedni komunikat zamiast tego.
Inne uwagi. SQL Injection, bez sensu klepiesz 3x ten sam select zamiast po prostu raz odpytać, hasło trzymaj w postaci zaszyfrowanej http://php.net/manual/en/function.password-hash.php albo co najmniej sha1() - odradzane.

[PHP] pobierz, plaintext 
$wynik_zapytania[0] != $login
[PHP] pobierz, plaintext 

To jest świetne. Skoro Select jest login = $login to spodziewasz się innego wyniku?

Ten post edytował viking 26.04.2015, 12:14:37