Czy takie logowanie jest bezpieczne? Opcje

[arosm]

Witam. Napisałem taki oto sobie system logowania i chciałem zasięgnąć opinii czy jest on w miarę bezpieczny. Mam tabelę Hasła z polami login, hasło,status (admin lub student), idsesji. Podczas logowania do tabeli zapisywany jest unikatowy Idsesji.

Plik authconfig.php

[PHP] pobierz, plaintext 
<?php
 
function connect_DB()
{
mysql_connect(&#092;"xxxx\", \"xxxxxxxx\", \"xxxx\") or die (\"Nie połączono z serwerem!\");
mysql_select_db(&#092;"xxxxxxxx\") or die (\"Nie wybrano bazy danych\");
};
 
function check_ID($ID)
{
$sql = &#092;"SELECT login FROM hasla WHERE Idsesji='$ID'\";
$result = mysql_query($sql) or die (&#092;"Nie pobrano wyników\");
$num = mysql_numrows($result);
return $num;
};
 
?>
[PHP] pobierz, plaintext 

Plik login.php

[PHP] pobierz, plaintext 
<?php
 
include_once(&#092;"authconfig.php\");
connect_DB();
$sql = &#092;"SELECT login, haslo FROM hasla WHERE login='$login' AND haslo='$pass'\";
$result = mysql_query($sql) or die (&#092;"Nie pobrano wyników\");
$num = mysql_numrows($result);
if ($num == 1) 
	{
		$string = crypt(rand());
		$idsesji = md5($string);
		$sql = &#092;"UPDATE hasla SET Idsesji = \"$idsesji\" WHERE login='$login' AND haslo='$pass'\";
		$sql_result = mysql_query($sql) or die (&#092;"NIE wykonano zapytania\");
		$stat = &#092;"Select status from hasla where login='$login' and haslo='$pass'\";
		$stat_result = mysql_query($stat) or die (&#092;"NIE wykonano zapytania\");
 
			while ($row = mysql_fetch_array($stat_result)) 
				{
					$status = $row[&#092;"status\"];
				}
		if ($status == 'pracownik') {
		header(&#092;"Location: dziekanat/index.php?PHPSESSIONID=$idsesji\");
		} else if ($status == 'student') {
		header(&#092;"Location: student/index.php?PHPSESSIONID=$idsesji\"); };
	} else if ($num == &#092;"0\") 
	{
		header(&#092;"Location: indexbl.php\");
	};
 
?>
[PHP] pobierz, plaintext 

Plik index.php

[PHP] pobierz, plaintext 
<?php
 
include_once(&#092;"../authconfig.php\");
connect_DB();
$numrows = check_ID($PHPSESSIONID);
if ($numrows == 1) {
 echo 'ZALOGOWANY ADMIN';
} else {
echo 'BRAK UPRAWNIEŃ';
}
 
?>
[PHP] pobierz, plaintext 

i dalej w tym pliku

[HTML] pobierz, plaintext 
<a href="index2.php?PHPSESSIONID=<? echo $PHPSESSIONID;  ?> " >LINK DO PODSTRONY</a>
[HTML] pobierz, plaintext 

plik index2.php

[PHP] pobierz, plaintext 
<?php
 
include_once(&#092;"../authconfig.php\");
connect_DB();
$numrows = check_ID($PHPSESSIONID);
if ($numrows == 1) {
echo ' ZALOGOWANY - ADMIN';
} else {
echo 'BRAK UPRAWNIEŃ';
}
 
?>
[PHP] pobierz, plaintext 

na tej stronie można to wypróbować.
login admina: admin hasło: dziekanat
login studenta: 1234 hasło: qwerty

Będę wdzięczny za wszelką krytykę i propozycje ulepszenia.

[Marusz]

Nie jest.

Wystarczy kliknac na ten adres:
http://www.uczelnia.bielsko.win.pl/dziekan...befd9a060ec6f2b

i juz sie jest zalogowanym.
Poza tym gdy zaloguje sie jako student, to wystarczy, ze zmienie w pasku przegladarki ze "/student/" na "/dziekanat/" i juz siedze w dziekanacie. Oczywiscie identyfikator sesji pozostaje bez zmian.

Ten post edytował Marusz 22.07.2005, 23:43:41