[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?

[Puciek]

Ale nie wyjasniles po jaka cholere dane do sesji cisniesz w md5.
Dalej, skąd sie bierze $login i $pass ? Pobierasz je z $_POST ? z $_GET ? z maszyny losujacej ? Czy poprostu w pelni radosnie jedziesz na register_globals ?
Stringi lepiej robic apostrofami, czyli

[PHP] pobierz, plaintext 
<?php
mysql_query( 'SELECT * FROM `USER` WHERE `login` = "'.$login.'" AND `haslo` = "'.$pass.'" LIMIT 0,1 ');
?>
[PHP] pobierz, plaintext 

Ten post edytował Puciek 11.09.2007, 15:17:36