$_SESSION i Uwierzytelnianie - pytanie Opcje

[MGreg]

Witam.
Męczę się nad bezpiecznym systemem uwierzytelniania i proszę ocenić, czy takie coś będzie bezpieczne:

[PHP] pobierz, plaintext 
<?php
session_start();
 
if(isset($_POST['login']) and isset($_POST['haslo']) and $_GET['action']=='zaloguj')
	{
	$query = mysql_query("select * from `users` where `login`='".mysql_real_escape_string($_POST['login'])."' and `haslo`='".mysql_real_escape_string(md5($_POST['haslo'])).'"');
	if(mysql_num_rows($query)>0)
	   {
	   $_SESSION['logged'] = 1;
	   $_SESSION['username'] = $_POST['login'];
	   }
	else
		{
		$_SESSION['logged']=0;
		{
	}
 
   if($_SESSION['logged']==1)
	  {
	   /* JAKIŚ TAM PANEL ADMINISTRACYJNY */
	  }
 
?>
[PHP] pobierz, plaintext 

Głównie chodzi mi o to, czy ktoś może zmienić wartość tablicy sesji $_SESSION['logged'] na 1 i tym samym zdobyć uprawnienia admina.

Ten post edytował MGreg 4.06.2008, 17:35:26

[MGreg]

Zobaczcie czy teraz jest lepiej. Dodałem sprawdzanie długości loginu i hasła, wstawiłem session_regenerate_id();, dodałem limit do zapytania, wywaliłem zapisywanie loginu do sesji i dodałem wylogowywanie.

[PHP] pobierz, plaintext 
<?php
session_start();
ob_start();
session_regenerate_id();
include('config.php');
pol_mysql();
 
if($_GET['action']=='logout')
	{
	if($_SESSION['logged']==1)
		{
		$_SESSION['logged']=0;
		echo "<b>zostałeś wylogowany<b>, jeśli nie zostaniesz przeniesiony na stronę logowania kliknij <a href=login.php>tutaj</a>";
		header("refresh: 5,login.php");
		}
	else
		{
		echo "nie byłeś zalogowany!";
		}
	}
 
if(!isset($_SESSION['logged'])){$_SESSION['logged']=0;}
 
if(isset($_POST['login']) and isset($_POST['haslo']))
   {	   
   if(strlen($_POST['login'])<25 and strlen($_POST['haslo'])<25)
	 {		   
	 $query = mysql_query("select `username`,`user_password` from `phpbb_users` where `username`='".mysql_real_escape_string($_POST['login'])."' and `user_password`='".mysql_real_escape_string(md5($_POST['haslo']))."' limit 1");
	   
	 if(mysql_num_rows($query)===1)
	   {			   
	   $_SESSION['logged'] = 1;	 
	   }
	 else
	  {
	  echo "Błędny login lub hasło";
	  $_SESSION['logged']= 0;
	  }
	 }		   
	 else
	  {
	  echo "Zbyt długi login lub hasło";
	  }	   
   }
	   
   if($_SESSION['logged']==1)
	 {	   
	 echo "Jakiś tam panel admina <a href=login.php?action=logout>Wyloguj</a>";		 
	 }	   
		   
   
 
if($_SESSION['logged']==0 and $_GET['action']!=='logout')
	{
	?>
	<form action="login.php" method=post>
	login <input type=text name=login><br>
	hasło <input type=password name=haslo><br>
	<input type=submit value=zaloguj>
	</form>
	<?
	}
		   
?>
[PHP] pobierz, plaintext 

Czy teraz jest lepiej?

Ten post edytował MGreg 5.06.2008, 10:27:15