LOGIN - Dove sbaglio ?

[-Daitarn-]

<?
require 'config/config.php'; // VARIABILI ACCESSO DB

// SALTO PAGINA
$jump_login = 'login.php';
$jump_logout = 'index.php';

session_start();

// LOGOUT
if($logout) {
    session_destroy();
    echo "<script language='JavaScript'>window.location='$jump_logout'</script>";
    exit();
}
// GIA LOGGATO
elseif(isset($HTTP_SESSION_VARS['access'])) {
    //$userid = $HTTP_SESSION_VARS['sblocco_userid']; // SBLOCCO UTENTE
}
// LOGIN
elseif(isset($HTTP_POST_VARS['username']) && isset($HTTP_POST_VARS['password'])) {
        
    // CERCO UTENTE
    $username = $_POST['username'];
    $password = $_POST['password'];
    $link = mysql_connect($dbhost, $dbuser, $dbpass);
    $query = "SELECT username, password FROM $dbcfg WHERE username='$username' AND password='$password'";
    $result = mysql_db_query($dbname, $query, $link);
    $found = @mysql_num_rows($result);
    
    if($found) { // SE UTENTE TROVATO
        $HTTP_SESSION_VARS['access'] = true;
        mysql_close($link);        
    } else {
        $error = "<font class='text'>[b]Errore: Nome Utente o Password non validi[/b]</font>

";
        mysql_close($link);
        include($jump_login);
        exit();
    }
} 
// NON LOGGATO
else {
    include($jump_login);
    exit();
}
?>

[Manuelandro]

complesso come login, troppe cose mescolate...

cmq che problema ti da?

[V1RuZ]

Originariamente inviato da Manuelandro
complesso come login, troppe cose mescolate...

cmq che problema ti da?

Infatti non piace nemmeno a me come è strutturato sto login...
Comunque, cominciamo con le osservazioni/domande

$logout che variabile è?

Se questo if non fa nulla perchè metterlo?
// GIA LOGGATO
elseif(isset($HTTP_SESSION_VARS['access'])) {
//$userid = $HTTP_SESSION_VARS['sblocco_userid']; // SBLOCCO UTENTE
}


Il redirect invece di farlo con:
echo "<script language='JavaScript'>window.location='$jump_logou t'</script>";
fallo con:
header("Location: $jump_logout");

Se è una versione di PHP superiore a 4.0.6 usa $_SESSION invece di $HTTP_SESSION_VARS e $_POST invece di $HTTP_POST_VARS

e soprattutto se trovi l'utente, setti la variabile di sessione access a true ma poi?