Forum PHP.pl

Witam, mam tu prościutki skrypt logowania na sesjach i cookie:

[PHP] pobierz, plaintext 
<?php
function logins(){
if(isset($_POST['password']) && isset($_POST['user'])) {
$user = addslashes(strtolower($_POST['user']));
$password = addslashes(md5(strtolower($_POST['password'])));
} else if(isset($_SESSION['user_login']) && isset($_SESSION['user_pass'])) {
$user = $_SESSION['user_login'];
$password = $_SESSION['user_pass'];
} else if(isset($_COOKIE['login'])) {
$r = explode("::", $_COOKIE['login']);
$user = $r[1];
$password = $r[0];
} else {
return false;
}
$query = "SELECT * FROM users WHERE password = '$password' AND user='$user'";
$sql = mysql_query($query);
$row = mysql_fetch_row($sql);
if(!mysql_num_rows($sql)) {
echo 'Niepoprawne Login lub Hasło';
return false;
}else{
$err = 'Zalogowany';
setcookie("login", "$password::$user", time()+24*360);
$_SESSION['user_login'] = $user;
$_SESSION['user_pass'] = $password;
return true;
}
}
 
 
 
if(logins()){
echo "<a href='?logout=yes'>wyloguj</a>";	
} else {
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>';
}
 
if(isset($_GET['logout']) && $_GET['logout']=="yes") {
session_destroy();
setcookie("login", '', time()-36);
header('location: '.$_SERVER['PHP_SELF'].'');
}
?>
[PHP] pobierz, plaintext 

Pytanie, czy warto wszystkie dane zapisać w sesji czy podtrzymywać logowania a dane wyciągać przez mysql_fetch_row a następnie je wyświetlać. Ogólnie skrypt to błazenada, ale na szybkiego jest. Nie wiem co tu jeszcze zmienić, oczywiście filtrowanie danych plus dodatkowe zabezpieczenia.

addslashes? użyj mysql_real_escape_string

I popracuj nad tabami, bo kompletnie nie można się rozczytać (tym bardziej, że klamerki trzymasz na końcu linijki, a nie jako osobna).

No a logowanie przecież musisz jakoś podtrzymać. Sesje się kłaniają. Chyba, że nie zrozumiałem pytania.