login in php

[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']?