... MySQL accepte ces trois syntaxes : #, -- et /* ... */ pour les commentaires, du moins la version 5 que j ai entre les mains...

1/ MySQL accepte ces trois syntaxes : #, -- et /* ... */ pour les commentaires, du moins la version 5 que j ai entre les mains.

2/ mysql_real_escape_string() double les apostrophes.

Je pense que tu mélange Oracle et MySQL

1/ le caractere de commentaire est # pour MySQL, -- pour Oracle
2/ mysql_real_escape_string() ajoute des / devant les caractères "dangereux" alors qu'avec Oracle, il suffit de doubler les '

Imagine la requête suivante:

<?php
$sql="SELECT * FROM users WHERE login='$login' AND pass='$pass'";

Rien ne t'empêche de saisir 'Pierre' --' comme valeur pour la variable $login, ce qui donnera la requête suivante à exécuter:

Code : Tout sélectionner

SELECT * FROM users WHERE login='Pierre' --' AND pass=''

Autrement dit, tu viens d'effectuer une injection SQL qui te permet de te connecter sans soumettre ton mot de passe (-- introduisant une ligne de commentaire en SQL)!

Si on enfermait $login dans la fonction mysql_real_escape_string(), on aurait

Code : Tout sélectionner

SELECT * FROM users WHERE login='Pierre'' --' AND pass=''

De plus, un vrai développeur désactive les magic quotes (qui vont d'ailleurs disparaître en PHP6...)

sql injection tu ne connais pas ?

j'en ai entendu parlé vaguement. Mais vu que je n'autorise pas les caractères ' et " dans mon formulaire, je pensais que cela empechait egalement les injections.

je vais me renseigner sur mysql_real_escape_string()

<?php
$sql="SELECT * FROM users WHERE login='$login' AND pass='$pass'";

sql injection tu ne connais pas ?

// on initialise la variable d'erreur
		$error = 0;
		// Vérification des données
		$check_pass = strcmp($_POST['password'], $_POST['password2']);
		if($check_pass != '0')
		{
			$error = 1;
			echo '<font class=\'error\'>'.$lang_password_wrong.'</font><br /><br />';
		}
		
		$check_mail = strcmp($_POST['mail'], $_POST['mail2']);
		if($check_mail != '0')
		{
			$error = 1;
			echo '<font class=\'error\'>'.$lang_mail_wrong.'</font><br /><br />';
		}
		
		// REGEX pour s'assurer que le login est correct
		strip_tags($_POST['login']);
		
		if(!preg_match('#^(?>[[:alnum:]]+)$#', $_POST['login']))
		{
			$error = 1;
			echo '<font class=\'error\'>'.$lang_login_unvalid.'</font><br /><br />';
		}
		
		
		// On interdit le < dans le mot de passe
		strip_tags($_POST['password']);
		
		if(!preg_match('<', $_POST['password']))
		{
			$error = 1;
			echo '<font class=\'error\'>'.$lang_pass_unvalid.'</font><br /><br />';
		}
		
		
		
		// REGEX pour s'assurer que l'e-mail est correcte
		strip_tags($_POST['mail']);
		
		if(!preg_match('`^[[:alnum:]]([-_.]?[[:alnum:]])*@[[:alnum:]]([-.]?[[:alnum:]])*\.([a-z]{2,4})$`', $_POST['mail']))
		{
			$error = 1;
			echo $lang_mail_unvalid;
		}
		
		
		
		// on vérifie que le login est unique
		$sql = 	"SELECT * 
				FROM USERS
				WHERE user_login = '".$_POST['login']."'";
		$query = mysql_query($sql) OR die('Erreur : '.mysql_error());
		$rows = mysql_num_rows($query);
		if($rows != 0)
		{
			$error = 1;
			echo '<font class=\'error\'>'.$lang_login_used.'</font><br /><br />';
		}
		
		// on vérifie que le mail est unique
		$sql = 	"SELECT * 
				FROM USERS
				WHERE user_mail = '".$_POST['mail']."'";
		$query = mysql_query($sql) OR die('Erreur : '.mysql_error());
		$rows = mysql_num_rows($query);
		if($rows != 0)
		{
			$error = 1;
			echo '<font class=\'error\'>'.$lang_mail_used.'</font><br /><br />';
		}
		
		// Si il y'a pas d'erreur, on fait l'enregistrement
		if($error == 0)
		{
			$sql = 	"INSERT INTO USERS
					VALUES ('', '".$_POST['login']."', '".md5($_POST['password'])."', '".$_POST['mail']."', '0', '0', '0', '".md5($_POST['login'])."')";
			$query = mysql_query($sql);
			
			
			
			$entete = "MIME-Version: 1.0\r\n";
			$entete .= "Content-type: text/html; charset=iso-8859-1\r\n";
			$entete .= "To: $to\r\n";
			$entete .= "From: $from\r\n";
			if(!mail($to, $sujet, $message, $entete))
				echo '<font class=\'error\'>'.$lang_mail_error.'</font>';
			else
				echo $lang_registered.'<br /><br />';
			
			
			exit();
		}