undefined variable sur une requête

**Seferight** · 29/05/2012, 18h29

Bonjour,

Je suis en train de créer un site internet en php, j'ai un problème sur ma page d'authentification. Lorsque je lance la page j'ai une erreur de variable non définit pour ma requête sql. Pourtant la requette est bien définit plus haut, je ne comprend pas

.
code en question

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
 
<html>
<head>
  <title>Authentification Base de Données</title>
</head>
<body>
<?php
  //connection au serveur
  $cnx = mysql_connect( "localhost", "root", "" ) ;
 
  //sélection de la base de données:
  $db  = mysql_select_db( "test" ) ;
 
 
  if (isset($_POST["pseudo"]))
  {
  $pseudo = $_POST["pseudo"] ;
  }
  if (isset($_POST["mdp"]))
  {
  $mdp = $_POST["mdp"] ;
  }
  if (isset($sql))
  {
  $sql = "Select * from membre where (pseudo='$pseudo') and (mdp='$mdp)'";
  }
 
 //exécution de la requête SQL:
	$requete = mysql_query($sql, $cnx) or die( mysql_error() ) ;
 
	//affichage des résultats, :
	if($requete)
	{
		echo("vous êtes maintenant connecté") ;
	}
	else
	{
		echo("la connexion à échouée") ;
	}
 
?>
</body>
</html>

L'erreur intervient sur $requete
"Notice: Undefined variable: sql in C:\wamp\www\tests\authentification_cible.php on line 27
Query was empty"

**Benjamin Delespierre** · 29/05/2012, 18h32

Heu...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
if (isset($sql))
  {
  $sql =

Tu vois pas comme un souci là

Là aussi y'a une erreur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

(mdp='$mdp)'

parenthèse du mauvais coté.

Ensuite, qu'est ce qu'il se passe selon toi si je mets "abc' and 1=1" comme mot de passe ?

**Seferight** · 29/05/2012, 18h42

Erreur d’inattention en effet, après suppression de l'isset sur le $sql j'obtient quand même une erreur:
( ! ) Notice: Undefined variable: pseudo in C:\wamp\www\tests\authentification_cible.php on line 22

( ! ) Notice: Undefined variable: mdp in C:\wamp\www\tests\authentification_cible.php on line 22

Code changer:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
 
<html>
<head>
  <title>Authentification Base de Données</title>
</head>
<body>
<?php
 
  $cnx = mysql_connect( "localhost", "root", "" ) ;
 
  //sélection de la base de données:
  $db  = mysql_select_db( "test" ) ;
 
  if (isset($_POST["pseudo"]))
  {
  $pseudo = $_POST["pseudo"] ;
  }
  if (isset($_POST["mdp"]))
  {
  $mdp = $_POST["mdp"] ;
  }
 
  $sql = "Select * from membre where (pseudo='$pseudo') and (mdp='$mdp')";
 
 
 //exécution de la requête SQL:
	$requete = mysql_query($sql, $cnx) or die( mysql_error() ) ;
 
	//affichage des résultats, pour savoir si l'insertion a marchée:
	if($requete)
	{
		echo("votre inscription à bien était effectuée") ;
	}
	else
	{
		echo("L'insertion à échouée") ;
	}
 
?>
</body>
</html>

Pour les codes d'accès, je suis en local donc je ne m'en suis pas occupé, j'ai pas rencontré de problème pour l'inscription d'un membre je doute que ça vienne de là

**ericd69** · 29/05/2012, 19h54

salut,

normal, si rien n'est donné en post ces variables ne sont pas défini vu que tu n'as pas mis de else sur les tests

d'un autre coté si aucun ou seul l'un des 2 n'est définis quel intérêt d'exécuter ta requête?

ton code est un trou de sécurité pour les attaques par injection sql... regarde comment te protéger grâce notamment aux techniques d'injection

et finalement oublie l'extension mysql, qui n'est plus maintenue, et utilise pdo à la place pour te connecter à mysql

**Benjamin Delespierre** · 31/05/2012, 11h11

La logique voudrait qu'on n'execute la requête SQL que si on a reçu les paramètres pseudo / mot de passe.

**redoran** · 31/05/2012, 11h32

Salam ; plus de ce qui été dit là dessus :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
<?php
....
 if (isset($_POST['verif_form'])) {  // si le formulaire est validé
         if((isset ($_POST['pseudo']) ) AND ........)) {
 
 
        try {
          tes instructions (requêtes et.....)
 
        }
       catch( Exception $e ){
	echo 'Erreur d\'affichage : ', $e->getMessage();
	}
      }
 
 
     }
 
 
?>

pour l'utilisation des condition http://php.net/manual/fr/control-structures.elseif.php
verfier aussi c'est tes variables sont bien transmis avec un var_dump Ok.

undefined variable sur une requête [MySQL]

PHP & Base de données

Vue hybride

Discussions similaires

Partager

Partager