Bonjour tous le monde !

Voilà j'ai créé mon propre espace membre. voici le script :

Une page normal .Qui verifie si l'utilisateur est enregistré :

<?php

session_start();

if(isset($_SESSION['user_name']))

{

echo "Bienvenue ".$_SESSION['user_name'];

echo "voici vos options, blablabla";

}else{

echo "formulaire d'identification";

}

?>

Script de vérification :il recupère les informations du formulaire d'identification :

<?php

if(isset($_POST['formlogin'])) //formlogin est le nom du bouton de validation du formulaire.

{

$login = htmlspecialchars($_POST['login']); // recuperation du login

$password = htmlspecialchars($_POST['password']); // recuperation du mot de pass

$res = mysql_fetch_array(mysql_query("SELECT * FROM user WHERE login = '".$login."'"));// éxecution de la requête sql

if($res['password'] == md5($password)) //vérification du mot de pass

{

$_SESSION['user_name'] = $res['login']; // si le mot de pass correspond ,on enregistre la session

}else{

echo "erreur d'identification"; //sinon on affichele message

}

}else{

echo "... on affiche le formulaire ici... ";

}

?>

La question est simple : est ce qu'il y a des failles dans ce script ?

et puis en passant es-il possible d'optimiser ce script ?( je pensse que oui ).

Salut,

optimisable, sans doute.

Pourquoi 2 pages pour cette verification ?

Faille de sécurité, ce n'en est pas une à proprement parler, mais je vois qqch qui me dérange : le SELECT *

Les raisons ici : http://www.php-astux.info/mysql-requetes-sql.php#requete-frequentes

(consulte le § sur le SELECT, et surtout les raisons en dessous)

Sinon au lieu des echo "..." tu peux utiliser echo '...' car le single quote est plus rapide à l'exécution que double quote (mais là je pinaille)