Forum PHP.pl

Witam, zrobiłem proste logowanie:

[PHP] pobierz, plaintext 
<?php
function alpha_numeric($str)
{
return ( ! preg_match("/^([-a-z0-9])+$/i", $str)) ? FALSE : TRUE;
}
 
function login(){
if(isset($_POST['user']) && isset($_POST['password'])) {
if (!$_POST['user'] || alpha_numeric($_POST['user'])==FALSE)
{
echo 'Login może zawierać jedynie liczby i litery';
} else {
$user = mysql_real_escape_string(strtolower($_POST['user']));
$password = md5(mysql_real_escape_string(strtolower($_POST['password'])));
$sql = mysql_query("SELECT * FROM users WHERE password = '$password' AND user = '$user' LIMIT 1");
$row = mysql_fetch_row($sql);
$rights = $row[4]; 
if(!mysql_num_rows($sql)) {
echo 'Niepoprawne Login lub Hasło';
return FALSE;
} else {
setcookie("login", $password, time()+24*360);
$_SESSION['rights'] = $rights;
$_SESSION['user_login'] = $user;
$_SESSION['user_pass'] = $password;
$_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
return TRUE;
}
}
}
 
if(isset($_COOKIE['login']) && !isset($_SESSION['rights'])){
$password = $_COOKIE['login'];
$sql = mysql_query("SELECT * FROM users WHERE password = '$password' LIMIT 1");
$row = mysql_fetch_row($sql);
$rights = $row[4];
$user = $row[2];
$password = $row[3]; 
if(mysql_num_rows($sql)) {
$_SESSION['rights'] = $rights;
$_SESSION['user_login'] = $user;
$_SESSION['user_pass'] = $password;
$_SESSION['ip'] = $_SERVER['REMOTE_ADDR'];
return TRUE;
}
}
}
 
login();
print_r($_SESSION);
 
function loged (){
if(isset($_SESSION['rights']) && $_SESSION['rights']=="1") {
echo "admin<br/>";
echo "<a href='?adminlogout=ok'>aaa</a>";
return TRUE;
} else if(isset($_SESSION['rights']) && $_SESSION['rights']=="2") {
echo "uzytkownik<br>";
echo "<a href='?adminlogout=ok'>aaa</a>";
return TRUE;
} else {
return FALSE;
}
}
 
?>
 
 
 
<?php
 
if($_GET['adminlogout']=="ok") {
session_destroy();
setcookie("login", '', time()-36);
header('location: logowanie.php');
}
 
if(!loged()){
echo '<form method="POST" action="">
<div>Podaj hasło</div>
<div><input type="text" name="user" /></div>
<div><input type="password" name="password" /></div>
<div><input type="checkbox" name="zapamietaj" /> - Zapamiętaj hasło</div>
<div><input type="submit" name="login" value="Zaloguj"/></div>
</form>';
}
 
?>
[PHP] pobierz, plaintext 

Ogólnie trochę namieszane. Dziś jednak cięzko mi idzie myślenie, więc nie wiem czy to ma sens :/

Proszę o jakieś wskazówki lub uwagi.

Z góry dziękuje

Ten post edytował potreb 9.05.2008, 14:56:05

Oprocz tego ze w sesji masz haslo i ze nie filtrujesz zmiennych w sql o tego ze polowa to sesje a druga polowa do cookie i na dodatek nie sprawdzasz czy dane z cookie sie zgadzaja z baza i sesje tez to jest ok...

P.S oczywiscie moge sie mylic ale ja mam calkiem inaczej to zrobione

Po co ci funkcja alpha_numeric? Lepiej skorzystać z http://pl.php.net/manual/en/function.ctype-alnum.php