login in php

[Joisberg]

Ragazzi ho un problema con un login... Se la sessione non viene inizializzata quando faccio il login, lui mi deve fare un redirect alla index... ma il fatto è che me lo fà sempre... non mi fà accedere alla pagina "riservata".
Vi posto il codice

<?php 
session_start(); 

session_regenerate_id(); 

include('config.php');

 $email_log = $_POST['email1'];

 $password = $_POST['password1'];



 if($email_log == "Email" || $password == "password123"){ 

              echo "Devi compilare tutti i campi";
 exit(); 

}

 if(!isset($_SESSION['login'] )){
 
           header("location:index.html");
 exit();
 
}else{ 

           if(isset($email_log) AND isset($password)){ 
                  $query = mysql_query("SELECT * FROM dati_utenti WHERE email='$email_log' AND password=MD5('$password') ") or die (mysql_error());

 $count = mysql_num_rows($query);
 
             if($count == 1){
                          $result = mysql_fetch_array($query);
                          $_SESSION['login'] = $result['nome']; 
           }else{ 

                    echo "Dati errati";
                exit(); 
 
}
           #se il login viene effettuato mi mostri questo
          if(isset($_SESSION['login'] )){
          echo "Benvenuto ".$_SESSION['login'];
         echo "<form action='logout.php' method='POST'><input type='submit' value='Esci' />
   </form>";
               } 
          } 
      } 
?>

[Alhazred]

Ci sono diverse cose che secondo me non vanno bene nel tuo codice:

$email_log = $_POST['email1'];
$password = $_POST['password1']; 

Visto che sui dati non fai nessun check (e invece dovresti farlo per evitare sql injection), tanto vale usare direttamente $_POST['email'] e $_POST['password'], l'assegnazione ti mangia solo memoria senza motivo.

if($email_log == "Email" || $password == "password123"){ 
    echo "Devi compilare tutti i campi";
    exit(); 
} 

Che succede se uno manda i campi vuoti? Il tuo codice non se ne accorge e invece dovrebbe mostrare lo stesso messaggio ed uscire.

if(isset($_SESSION['login'] )){
    echo "Benvenuto ".$_SESSION['login'];
    echo "<form action='logout.php' method='POST'><input type='submit' value='Esci' /></form>";
} 

Non te la mostra questa cosa se il login va a buon fine e quindi inizializzi $_SESSION[login']?

[Joisberg]

if($email_log == "Email" || $password == "password123"){ 
          echo "Devi compilare tutti i campi"; 
          exit(); 
} 

Ho fatto questo perchè nell'html ho messo questo

<label class="login">Login</label>
 
<form action="login.php" method="POST"> 

<center> 

<hr style="width:170px; background:#cccccc;"/>

 <input type="text" name="email1" class="em" value="Email" onblur="if(this.value == '') { this.value='Email'}" onfocus="if (this.value == 'Email') {this.value=''}" />


 

<input type="password" name="password1" class="ps" value="Password123" onblur="if(this.value == '') { this.value='Password123'}" onfocus="if (this.value == 'Password123') {this.value=''}" />

 <input type="submit" value="Accedi" class="accedi" /> 

</center>

</form>

Ma il vero problema è che se faccio una pulizia dei cookie e delle sessioni, quando faccio il login non mi fà accedere ma mi riporta sempre alla index (perchè io ho impostato che se la sessione non è inizializzata mi fà un redirect. In parole povere non mi attiva la sessione! )

[Alhazred]

Originariamente inviato da Joisberg
Ho fatto questo perchè nell'html ho messo questo [...]

I controlli Javascript lasciano il tempo che trovano, se chi è sulla pagina disabilita Javascript quei controlli non vengono eseguiti e si possono mandare anche campi vuoti.
Tu dirai "chi è quel fesso che dovrebbe fare una cosa simile?", chi per esempio vuole forzare il sistema, se fai solo controlli Javascript gli lasci vita facile per farti dei bei danni.

Ma il vero problema è che se faccio una pulizia dei cookie e delle sessioni, quando faccio il login non mi fà accedere ma mi riporta sempre alla index (perchè io ho impostato che se la sessione non è inizializzata mi fà un redirect. In parole povere non mi attiva la sessione! )

Dovresti postare tutto il codice coinvolto, non abbiamo la sfera di cristallo per sapere cosa fai altrove e magari è lì che si trova il problema.

[Joisberg]

<?php

session_start();
session_regenerate_id();
#session_register('login');

include('config.php');

$email_log = $_POST['email1'];
$password = $_POST['password1'];
#$id = $_SESSION['login'];

if($email_log == "Email" || $password == "password123"){

header("location:login.html");
exit();
}

if(!isset($_SESSION['login'] )) {

header("location:index.html") ;
exit();

}else{

if(isset($email_log) AND isset($password)){

$query = mysql_query("SELECT * FROM dati_utenti WHERE email='$email_log' AND password=MD5('$password') ") or die (mysql_error());

$count = mysql_num_rows($query);

if($count == 1){

$result = mysql_fetch_array($query);
$_SESSION['login'] = $result['nome']." ".$result['cognome'];



}else{

header("location:login.html");
exit();
}

if(isset($_SESSION['login'] )){
echo "Benvenuto <b style='color:#143d66;'>".$_SESSION['login']."[/b]";
echo "<form action='logout.php' method='POST'><input type='submit' value='Esci' /></form>";
}
}
}
?>

[Joisberg]

Andiamo qualcuno che mi aiuti