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

[PHP] pobierz, plaintext 
<?php
if(!isset($_SESSION['login']){
   $_SESSION['login'] = false;
}
 
function check_user($login, $pass)
{
 
 
	if(empty($login) OR empty($pass))
	{
	  echo 'Nie podałeś Loginu i/lub Nicka';
	  return false;
	}
	else
	{
	  $login = mysql_escape_string($login);
	  $pass = md5($pass);
 
	  $dane = mysql_query("SELECT * FROM `USER` WHERE `login` = '".$login."' AND `haslo` = '".$pass."' LIMIT 0, 1" );
 
	  if(mysql_num_rows($dane) == 1)
	  {		
		$_SESSION['login'] = true;
 
		echo $login.' poprawnie sie zalogował';
		return true;
	  }
	  else
	  {
		echo 'Login i/lub Hasło są nieprawidłowe!';
		return false;
	  }
 
  }
 
}
?>
[PHP] pobierz, plaintext 

mniej wiecej...

Ten post edytował cornholio666 11.09.2007, 16:01:34