[PHP]Czy jest to wystarczające zabezpieczenie? - autoryzacja Opcje

[b_chmura]

Witam

Chciałbym się zapytać was ludzi z doświadczeniem i pomysłem czy takie zabezpieczenie jest wystarczające:

[PHP] pobierz, plaintext 
<?php
//logowanie - jeśli login i hasło są poprawne tworzy się sesja zakodowanego przez md5 loginu 
i hasla
function loguj($login, $pass) 
{
  $spr = mysql_query("SELECT * FROM `USER` WHERE `login` = 'login' AND `haslo` = '$pass'" LIMIT 0,1);
 
  if(mysql_num_rows($spr) == 1)
  {
	$_SESSION['login'] = md5($login.$pass);
	return true;
  }
  else
  {
	return false;
  }
}
//sprawdzanie czy user jest zalogowany - jeśli dane wyciągnięte i zakodowane przez md5 zgadzają sie z sesją wszystko jes
t ok
function spr_usr()
{
  $query = mysql_query("SELECT * FROM `USER` WHERE `login` = '$login' AND `haslo` = '$pass'");
 
  while($user  = mysql_fetch_array($query))
  {
	if(md5($user['login'].$user['haslo']) = $_SESSION['login'])
	{
	  return true;
	}
	else
	{
	  return false;
	}
  }
}
?>
[PHP] pobierz, plaintext 

przejdzie to?

[cornholio666]

ad4
bo są to osobne funkcje

Po co ci aż dwa zapytania żeby sprawdzic czy uzytkownik istnieje? nie da sie jednym ?

Tutaj w pętli haszujesz login i haslo, czyli jak baza bedzie miala 100000 userów to troche niewydajne bedzie.

[PHP] pobierz, plaintext 
<?php
if(md5($user['login'].$user['haslo']) == $_SESSION['login'])
?>
[PHP] pobierz, plaintext 

Z kodu wynika ze trzymasz hasło w bazie w postaci jawnej a to duzy błąd