PHP

votre avis sur la sécurité de ce script...

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : votre avis sur la sécurité de ce script...

lilougirl8

Bonjour,

Pourriez vous me dire ce que vous pensez de la sécurité de ce script que j'ai créer qui permet de se loguer et protéger mes pages.

Le script de connexion

Code :

<?php
session_start();
$_SESSION['logged'] = 0;
include("".$_SERVER['DOCUMENT_ROOT']."/script_php/fonctions_php/fonction_racine.php" );
if(!empty($_POST['login']) && (!empty($_POST['mdp']))) {
require("".$racine."/securite/sql/connexion_sql.class.php" );
/*================ CONNEXION A LA BDD =============== */
$connexion = new connexion_sql();
$connexion = $connexion->connexion_PDO();
/*=======================================================*/
$login = trim($_POST['login']);
$pass = mhash(MHASH_SHA256, trim($_POST['mdp']));
$page_precedente = $_POST['page_precedente'];
$requete_prepare_log = $connexion->prepare("SELECT login, pass, droit FROM admin_membres WHERE login = :login" );
$requete_prepare_log->execute(array(':login', => $login));
if($utilisateur = $requete_prepare_log->fetch(PDO::FETCH_OBJ)) {
if($pass == $utilisateur->pass) {
$_SESSION['user'] = $utilisateur;
$_SESSION['ip_client'] = $_SERVER['REMOTE_ADDR'];
$_SESSION['HTTP_USER_AGENT'] = mhash(MHASH_SHA256, $_SERVER['HTTP_USER_AGENT']);
$_SESSION['logged'] = 1;
$_SESSION['iniated'] = true;
$requete_prepare_log->closeCursor();
header("Location:".$page_precedente."" );
die();
}
else {
$requete_prepare_log->closeCursor();
header("Location:".$racine_relatif."/admin/login.php?erreur=1" );
die();
}
}
else {
$requete_prepare_log->closeCursor();
header("Location:".$racine_relatif."/admin/login.php?erreur=1" );
die();
}
}
else {
header("Location:".$racine_relatif."/admin/login.php?erreur=2" );
die();
}
?>

La fonction a mettre en debut de chaque page

Code :

function page_securite() {
ini_set('session.use_trans_sid', 0);
session_start();
if (!isset($_SESSION['initiated'])) {
session_regenerate_id();
$_SESSION['initiated'] = true;
}
if(isset($_SESSION['logged'])) {
if($_SESSION['logged'] == 1) {
if(isset($_SESSION['ip_client']) && ($_SESSION['ip_client'] == $_SERVER['REMOTE_ADDR'])) {
if (isset($_SESSION['HTTP_USER_AGENT'])) {
if ($_SESSION['HTTP_USER_AGENT'] == mhash(MHASH_SHA256, $_SERVER['HTTP_USER_AGENT'])) {
if(isset($_SESSION['user'])) {
return true;
}
}
}
}
}
}
return false;
}

Sur chaques pages

Code :

if(!page_securite()) {
header(Page d'erreur);
die();
}
else {
On affiche la page
}

Je vais ajouter en plus de cela une gestion avec cookies qui me permettre de me souvenir de l'utilisateur.

Merci d'avance pour vos avis et critiques

Message édité par lilougirl8 le 11-02-2008 à 11:32:13

Publicité

yellu

Sans juger la synthaxe et l'ergonomie du code, je trouve que la volonté de consolider la SESSION PHP via l'ip et user-agent est une bonne chose.

lilougirl8

merci c'est sympa, je compte rajouter un compteur d'attaque en cas de mauvais mot de passe.

Par contre, je veux mettre un en place un système de cookie comme indiqué plus haut, comment ça se passe concrètement quand je créer le cookie?

Voila ce que je pensai faire :

Lorsque l'utilisateur se connect et que tout c'est bien passé : je créer le cookie login, le cookiee mot de passe,

- Quand l'utilisateur revien sur la page, si le cookie login et pass existe, je vérifie qu'il soit valide (en faisant une requete) et je le connecte automatiquement.

C'est ça qu'il faut faire?? j'ai l'impression que il faudrait peut etre augmenter la sécurité à ce niveau la

NewsletTux

<Insérez ici votre vie />

pas forcément ... L'UA est facilement modifiable et l'IP peut être masquée ... Donc imo, cela apporte certes un plus mais qui n'est pas synonyme de palier de sécurité.

D'autre part, perso je ne vois pas l'intérêt de faire un SELECT pour rapatrier le mot de passe : il vaut mieux imo le laisser dans la BDD (d'autant plus qu'à en lire le script, il n'a pas l'air d'être vérifié à l'heure actuelle)

---------------
NewsletTux - outil de mailing list en PHP MySQL

lilougirl8

Voilà j'ai rectifier c'est bien ça que tu ma conseillé??

Code :

$requete_prepare_log = $connexion->prepare("SELECT login, droit FROM membres WHERE login = :login AND pass = :pass" );
$requete_prepare_log->execute(array(':login' => $login, ':pass' => $pass));
if($utilisateur = $requete_prepare_log->fetch(PDO::FETCH_OBJ)) {
$requete_prepare_log->closeCursor(); //On ferme le curseur
$_SESSION['user'] = $utilisateur;
$_SESSION['ip_client'] = $_SERVER['REMOTE_ADDR'];
$_SESSION['HTTP_USER_AGENT'] = mhash(MHASH_SHA256, $_SERVER['HTTP_USER_AGENT']);
$_SESSION['logged'] = 1;
$_SESSION['iniated'] = true;
$duree_cookie = time() + 31536000; // valable un an
setcookie(); //Cookie Login
setcookie(); // Cookie Mot de passe
header("Location:".$page_precedente."" );
die();
}
else {
$requete_prepare_log->closeCursor();
header("Location:".$racine_relatif."/admin/login.php?erreur=1" );
die();
}
}
else {
header("Location:".$racine_relatif."/admin/login.php?erreur=2" );
die();
}

EDIT : MERCI NazzTazz, erreur Corrigée

Message édité par lilougirl8 le 11-02-2008 à 13:01:24

FORUM HardWare.fr

Programmation

PHP

votre avis sur la sécurité de ce script...

Sujets relatifs
Script bash	executer script
Modifier un script de compte a rebour	Copie de sécurité
Cherche un script pour afficher un fichier csv	Faille de sécurité HTML
Mon site, votre avis?	htaccess/htpasswd => sécurité fiable?
Script Outlook	ya t-il un remède à l'erreur 405 pour script CGI ?
Plus de sujets relatifs à : votre avis sur la sécurité de ce script...

Page générée en 0.065 secondes