Sistema di login e sessioni (giusto?)

[sko]

//in ogni pagina metto:
$cookietime = 5000;
session_set_cookie_params($cookietime);
session_start();
if (isset ($_SESSION[us])) echo "ciao $_SESSION[us]";


//Nel login una volta riconosciuto l utente
//(visto che ci siamo approfitto x sapere se è giusto il riconoscimento)

$conta = mysql_query("SELECT COUNT(*) FROM utenti WHERE username='$user' && password='$pass'") or die(mysql_error());
$ris = mysql_fetch_array($conta);
$result = mysql_query($query, $db);
$row = mysql_fetch_array($result);

//se row[0]= uno allora ha trovato l utente
//quindi

$expireTime = 60*5; // La sessione dura 5 Minuti

session_set_cookie_params($expireTime);
session_start();
$_SESSION['us']=$user ;

// nel logout

session_start();
$_SESSION = array();
// Infine distrugge la sessione.
session_destroy(); 

ho voluto appositamente dare piu vita alla sessione anche alla chiusra del cookie vista la natura del sito non mi serviva tutta quella sicurezza a discapito di far ogni volta il login.

sono agli inizi inizi del php... vorrei sapere se il codice va' bene ....

ciao e grazie

[sko]

up...


nessuno mi dice nulla??

non vorrei fare errori grossolani ... almeno nel login

[boomboom69]

non ti riesco a seguire molto cmq qui un errore c'è

$conta = mysql_query("SELECT COUNT(*) FROM utenti WHERE username='$user' && password='$pass'") or die(mysql_error());
$ris = mysql_fetch_array($conta);
$result = mysql_query($query, $db);
$row = mysql_fetch_array($result);

// corretto

$result = mysql_query("SELECT 1 FROM utenti WHERE username='$user' AND password='$pass'") or die(mysql_error());
$row = mysql_num_rows($result);

ma $user e $pass dove le prendi ogni volta?

[sko]

Originariamente inviato da boomboom69
non ti riesco a seguire molto cmq qui un errore c'è

$conta = mysql_query("SELECT COUNT(*) FROM utenti WHERE username='$user' && password='$pass'") or die(mysql_error());
$ris = mysql_fetch_array($conta);
$result = mysql_query($query, $db);
$row = mysql_fetch_array($result);

// corretto

$result = mysql_query("SELECT 1 FROM utenti WHERE username='$user' AND password='$pass'") or die(mysql_error());
$row = mysql_num_rows($result);

ma $user e $pass dove le prendi ogni volta?

grazie per la correzione!ora provo quel codice..

user e pass la prima volta li prendo dal form di login.. x le volte successive controllo se il campo $_session[us] esiste!

[sko]

Scusate se uppo ancora... ma a me funziona ..solo che nn vorrei star facendo dei grossi errori di sicurezza!
almeno il login lo vorrei tranquillo

[pgm]

ciao sko

ti rimando a questa discussione
http://forum.html.it/forum/showthrea...hreadid=440186

dagli un'occhiata