kokotchY
Elite
Moi j'addslashes toujours les valeurs avant l'insertion des données !
$login = (isset($_GET['login']))?addslashes($_GET['login']):'';
$confirm = (isset($_GET['confirm']))?addslashes($_GET['confirm']):'';
c'est toujours plus simple
aussi pour la petite info, md5 hash le password ce qui le rend par définition indéchiffrable !
La seule chose que l'on peut faire c'est de hasher le password donné par l'utilisateur et le comparer
Pour ceci :
confirm.php
c'est un peu bete de faire un mysql_fetch_array qui est quand même assez lourd par rapport a un mysql_fetch_row ou encore un mysql_result !
$nb = mysql_result($req,0,'nb');
$nb contiendra la même chose que $data['nb]
Au fur et a mesure que je regarde le code, je trouve des petites erreurs
confirm.php :
mysql_query("UPDATE membres SET `confirm`='' WHERE `login` ='$login'") or die('erreur : '.mysql_error());
$login n'est pas défini donc ca ne marchera pas
auth2.php :
c'est inutile de tout sélectionner dans la base de donnée si ce n'est que pour prendre le password :
$sql = "SELECT password FROM membres_tbl WHERE login='$_POST[login]'";
$req = mysql_query($sql) or die('erreur:' .mysql_error());
$pass = mysql_result($req,0,'password');
espace.php :
ARG
session_register(login);
ne jamais utiliser ca :/ c'est obsolète au possible !
$_SESSION['login'] = 'login';
Sinon je n'ai rien trouvé d'autre :roll:
$login = (isset($_GET['login']))?addslashes($_GET['login']):'';
$confirm = (isset($_GET['confirm']))?addslashes($_GET['confirm']):'';
Code:
$sql = "SELECT COUNT(*) as nb FROM membres WHERE login = '$login' AND confirm = '$confirm'";
aussi pour la petite info, md5 hash le password ce qui le rend par définition indéchiffrable !
La seule chose que l'on peut faire c'est de hasher le password donné par l'utilisateur et le comparer
Pour ceci :
confirm.php
Code:
$sql = 'SELECT COUNT(*) as nb FROM membres WHERE login = "'.addslashes($_GET['login']).'" AND confirm = "'.addslashes($_GET['confirm']).'"';
$req = mysql_query($sql) or die(mysql_error());
$data = mysql_fetch_array($req);
$nb = mysql_result($req,0,'nb');
$nb contiendra la même chose que $data['nb]
Au fur et a mesure que je regarde le code, je trouve des petites erreurs
confirm.php :
mysql_query("UPDATE membres SET `confirm`='' WHERE `login` ='$login'") or die('erreur : '.mysql_error());
$login n'est pas défini donc ca ne marchera pas
auth2.php :
c'est inutile de tout sélectionner dans la base de donnée si ce n'est que pour prendre le password :
$sql = "SELECT password FROM membres_tbl WHERE login='$_POST[login]'";
$req = mysql_query($sql) or die('erreur:' .mysql_error());
$pass = mysql_result($req,0,'password');
espace.php :
ARG
session_register(login);
ne jamais utiliser ca :/ c'est obsolète au possible !
$_SESSION['login'] = 'login';
Sinon je n'ai rien trouvé d'autre :roll: