[Sécurité] Mauvaise requête + session

[pod1978]

Bonjour a tous,

J'ai mis en place un espace membre dans lequel lorsque la personne s'identifie, elle est dirigée vers sa page d'infos qu'elle peut modifier.

Donc j'ai deux tables sous mysql qui sont en relation:

le login de la table tbl_user et le mail de la table gite.

Le probleme, avec mon code il affiche tous les enregistrements qui ont comme requete la relation entre login de tbl_user et le mail de la table gite.
Alors je veux seulement les enregistrements de la personne identifiée et non celles de toute la base

Je vous montre mon code:

Code :

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
44
45
46
47
48
49
50
51
52
<?
$base = mysql_connect (', , );
mysql_select_db (, $base);
 
 
 
 
if(isset($_POST) && !empty($_POST['login']) && !empty($_POST['pass'])) {
  extract($_POST);
  // on recupère le password de la table qui correspond au login du visiteur
  $sql = "select pwd from tbl_user where login='".$login."'";
  $req = mysql_query($sql) or die('Erreur SQL !<br>'.$sql.'<br>'.mysql_error());
 
  $data = mysql_fetch_assoc($req);
 
  if($data['pwd'] != $pass) {
    echo '<p>Mauvais login / password. Merci de recommencer</p>';
    include('login.htm'); // On inclut le formulaire d'identification
    exit;
  }
  else {
    session_start();
    $_SESSION['login'] = $login;
 
    echo 'Vous etes bien logué';
 
  }   
}
else {
  echo '<p>Vous avez oublié de remplir un champ.</p>';
   include('login.htm'); // On inclut le formulaire d'identification
   exit;
}
 
        $cnx = mysql_connect( '', , ) ;
 
   :
    $db = mysql_select_db( ) ;
 
     $sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio=tbl_user.login" ;
    //exécution de la requête:
    $requete = mysql_query( $sql, $cnx ) ;
 
 
    while( $result = mysql_fetch_object( $requete ) )
 
    {
       echo("<table  ><tr><td class=texte_requete4 >".$result->login."&nbsp;</td><td width=650 class=texte_requete4><b>".$result->nomGite."</b>&nbsp;//".$result->regionGite."&nbsp;//".$result->dptGite."<br>".$result->nomProprio."&nbsp;//".$result->mailProprio."</td><td>&nbsp;</td><td class=texte_requete4> <a href=\"requete3.php?id=".$result->id."\">modifier</a></td></tr>\n") ;
 
    }
 
  ?>

J'espere avoir été assez clair

Merci de votre aide

[monsieurjean]

Code :

 $sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio=tbl_user.login" ;

il manque une contrainte sur l'utilisteur, aucune variable dans la requete ??

[pod1978]

Je ne sais pas coment faire correpondre le mail de l'utilisateur avec les enregistrements qui lui correspondent!

Code :

$sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio=tbl_user.login" ;

J'ai essayé de mettre:

Code :

 $sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio=tbl_user."$_SESSION['login']"" ;

Mais cela ne fonctionne pas.

J'ai essayé de me renseigner un peu partout mais je suis vraiement coincé

[Amara]

Code :

 $sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio=tbl_user." . $_SESSION['login'];

Peut-être un peu mieux déjà :

[monsieurjean]

si mailProprio est un champ contenant une valeur identique au login :

Code :

1
2
 
$sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio='"$_SESSION['login']"'" ;

[pod1978]

J'ai essayé la requete suivante:

Code :

$sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio=tbl_user." . $_SESSION['login'];

Mais cel ne fonctionne pas, ca me met:

Code :

Warning: mysql_fetch_object(): supplied argument is not a valid MySQL result resource in /var/www/free.fr/3/f/pod1978/annuaire/membre/verif.php on line 49

La ligne 49 correspoind a celle qui commence par while

Code :

1
2
3
4
5
6
7
8
9
10
11
$sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio=tbl_user." . $_SESSION['login'];
    //exécution de la requête:
    $requete = mysql_query( $sql, $cnx ) ;
 
    //affichage des données:
    while( $result = mysql_fetch_object( $requete ) )
 
    {
       echo("<table  ><tr><td class=texte_requete4 >".$result->login."&nbsp;</td><td width=650 class=texte_requete4><b>".$result->nomGite."</b>&nbsp;//".$result->regionGite."&nbsp;//".$result->dptGite."<br>".$result->nomProprio."&nbsp;//".$result->mailProprio."</td><td>&nbsp;</td><td class=texte_requete4> <a href=\"requete3.php?id=".$result->id."\">modifier</a></td></tr>\n") ;
 
    }

Concernant la requete:

Code :

$sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio='"$_SESSION['login']"'" ;

J'ai aussi testé mais cela ne fonctionne pas non plus, ca me met cela comme message d'erreur:

Code :

Parse error: parse error, unexpected T_VARIABLE in /var/www/free.fr/3/f/pod1978/annuaire/membre/verif.php on line 44

Et lea ligne 44 correspond a cette requete


Merci de votre aide

[Hervé Saladin]

Envoyé par JWhite

Code :

 $sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio=tbl_user." . $_SESSION['login']

;


Peut-être un peu mieux déjà :

Non !

Envoyé par monsieurjean

si mailProprio est un champ contenant une valeur identique au login :

Code :

$sql = "SELECT * FROM gite,tbl_user  WHERE mailProprio='"$_SESSION['login']"'" ;

Non plus !!!

Bon, si tu fais une selection sur deux tables il faut que tu fasses une jointure ET une condition sur le login
en gros :

Code :

$sql = "SELECT * FROM gite, tbl_user WHERE gite.mailProprio=tbl_user.login AND tbl_user.login='".$_SESSION['login']."'";

[Amara]

Exact Hervé je m'étais bien embrouillé sur le coup

[pod1978]

Merci la 1ere étape fonctionne tres bien.

Maintenant en fait mon souci viens que lorsque ses enregistrements pparaissent , j'ai un line modifier qui m'envoie vers une autre page.
Mais j'ai le meme souci que tout a l'heure, la requete ne marche pas alors que j'ai repris le meme.

Je vous montre mon code de ma page requete3.php

Code :

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
<?php
 session_start();
 ?>
 <html>
  <head>
    <title>modification de données en PHP :: partie2</title>
 
<link href="../sources/style_index.css" rel="stylesheet" type="text/css">
</head>
<body>
  <?php
  //connection au serveur:
  $cnx = mysql_connect( ', , ) ;
 
  //sélection de la base de données:
  $db = mysql_select_db( ) ;
 
  //récupération de la variable d'URL,
  //qui va nous permettre de savoir quel enregistrement modifier
  $login  = $_GET["login"] ;
 
  //requête SQL:
$sql = "SELECT * FROM gite, tbl_user WHERE gite.mailProprio=tbl_user.login AND WHERE login = ".$login ;
 
  //exécution de la requête:
  $requete = mysql_query( $sql, $cnx ) ;
 //affichage des données:
  if( $result = mysql_fetch_object( $requete ) )
  {
 
  ?>  
 
 
  <form name="insertion" action="requete4.php" method="POST">
  <input type="hidden" name="id" value="<?php echo($id) ;?>">
  <table border="0" align="center" cellspacing="2" cellpadding="2">
  <tr><td class="titre_titre"><?php echo($result->id) ;?></td></tr>
    <tr align="center">
      <td class="texte_formulaire">nomGite</td>
      <td><input type="text" size="80" name="nomGite" value="<?php echo($result->nomGite) ;?>"></td>
    </tr>
    <tr align="center">
      <td class="texte_formulaire">descriptionGite</td>
      <td><input type="text"size="80" name="descriptionGite" value="<?php echo($result->descriptionGite) ;?>"></td>
    </tr>
    <tr align="center">
      <td class="texte_formulaire">regionGite</td>
      <td><input type="text" size="80"name="regionGite" value="<?php echo($result->regionGite) ;?>"></td>
    </tr>
    <tr align="center">
      <td class="texte_formulaire">dptGite</td>
      <td><input type="text" size="80" name="dptGite" value="<?php echo($result->dptGite) ;?>"></td>
    </tr>
    <tr align="center">
      <td class="texte_formulaire">typeGite</td>
      <td><input type="text" size="80"name="typeGite" value="<?php echo($result->typeGite) ;?>"></td>
    </tr>
 
     <tr align="center">
      <td class="texte_formulaire">capaciteGite</td>
      <td><input type="text" size="80"name="nomProprio" value="<?php echo($result->capaciteGite) ;?>"></td>
    </tr>
	 <tr align="center">
      <td class="texte_formulaire">Nom</td>
      <td><input type="text" size="80"name="nomProprio" value="<?php echo($result->nomProprio) ;?>"></td>
    </tr> 
    <tr align="center">
      <td class="texte_formulaire">mailProprio</td>
      <td><input type="text" size="80"name="mailProprio" value="<?php echo($result->mailProprio) ;?>"></td>
    </tr>  <tr align="center">
      <td class="texte_formulaire">telProprio</td>
      <td><input type="text" size="80"name="telProprio" value="<?php echo($result->telProprio) ;?>"></td>
    </tr>
    <tr align="center">
      <td class="texte_formulaire">siteProprio</td>
      <td><input type="text" size="80"name="siteProprio" value="<?php echo($result->siteProprio) ;?>"></td>
 
 
    <tr align="center">
      <td colspan="2"><input type="submit" value="modifier"></td>
    </tr>
  </table>
</form>
  <?php
  }//fin if 
  ?>
</body>
</html>

Il me remet ce message d'erruer:

Code :

Warning: mysql_fetch_object(): supplied argument is not a valid MySQL result resource in /var/www/free.fr/3/f/pod1978/annuaire/membre/requete3.php on line 28

Merci de votre aide

[Hervé Saladin]

on ne met qu'une seule fois le where
c'est de la forme

Code :

1
2
3
select deschamps
from destables
where booleen

donc la requete suivante est mauvaise :

Code :

SELECT * FROM gite, tbl_user WHERE gite.mailProprio=tbl_user.login AND WHERE login = ...

il faut mettre :

Code :

SELECT * FROM gite, tbl_user WHERE gite.mailProprio=tbl_user.login AND tbl_user.login = ...

tout simplement

PS : à mon avis tu as plus besoin de l'aide du forum SQL que de celle du forum PHP

[pod1978]

Excuse moi je suis allé trop vite pour faire la requete mais il semble que cela ne fonctionne toujourrs pas

Tjrs le meme message:

Code :

Warning: mysql_fetch_object(): supplied argument is not a valid MySQL result resource in /var/www/free.fr/3/f/pod1978/annuaire/membre/requete3.php on line 28

Voici le code et la ligne 28 est celle qui commence par while:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<?php
  //connection au serveur:
  $cnx = mysql_connect( '', , ) ;
 
  //sélection de la base de données:
  $db = mysql_select_db( ) ;
 
  //récupération de la variable d'URL,
  //qui va nous permettre de savoir quel enregistrement modifier
  $login  = $_GET["login"] ;
 
  //requête SQL:
$sql = "SELECT * FROM gite, tbl_user WHERE gite.mailProprio=tbl_user.login AND login = ".$login ;
 
  //exécution de la requête:
  $requete = mysql_query( $sql, $cnx ) ;
 //affichage des données:
  if( $result = mysql_fetch_object( $requete ) )
  {
 
  ?>

En fait ds ce cas présent, une propriétaire a deux enregistrements donc je ne sais pas si c bon que la requete se termine pae login=".$login" car ca sera deux fois le meme mail. Faut peut etre passer par un autre identifiant mais je ne sais pas trop

Merci

[pod1978]

Je peux vous monter l'url:

http://pod1978.free.fr/annuaire/membre/login.htm

le login:

martine.deroo@wanadoo.fr

le mot de passe:
boubou

Merci de votre aide

[Amara]

Peut-être avec des quotes :

Code :

$sql = "SELECT * FROM gite, tbl_user WHERE gite.mailProprio=tbl_user.login AND login = '". $login . "'" ;

[billoum]

Envoyé par JWhite

Peut-être avec des quotes :

Code :

$sql = "SELECT * FROM gite, tbl_user WHERE gite.mailProprio=tbl_user.login AND login = '". $login . "'" ;

+1
Quand tu as cette erreur :

Code :

supplied argument is not a valid MySQL result resource

Toujours faire :

Code :

$requete = mysql_query( $sql, $cnx ) or die(mysql_error());

et un echo de ta requete pour avoir une meillleur idée de l'erreur qui s'est produite