SÉCURISATION DE FORMULAIRE

Commentaire de ultimacsplayer le 16/06/2010 07:31:33

En fait, a la place de

#
# if($_POST['pseudo']=="Chris" && $_POST['pass']=="test"){
# echo "Bienvenue ".$_POST['pseudo'].", votre mot de pass est ".$_POST['pass']."!";
# }

c'est là qu'on met le code qui se connecte a la base de donnée et qui regarde si le pseudo et pass est correct.

Commentaire de destinyfr le 16/06/2010 12:31:13

Salut,

Pourquoi réinventer la chose (en bien moins performant d'ailleurs), alors qu'il existe déjà plein de fonctions pour sécuriser les données ? (Par exemple : filter_*).
De plus, ton code n'est pas vraiment adapté pour une réutilisation...

Bye ;)

Commentaire de ultimacsplayer le 16/06/2010 15:49:52

Merci pour vos informations, et surtout pour la derniere requete de Arto, il est clair que l'injection sql ne se limite pas à bannir certains champs.

Je pensais bien faire en partageant un bout de code anti injection, et grace à vous, il y a une meilleur solution que celle que j'ai proposé!

Commentaire de masternico le 21/06/2010 10:15:23

Perso, je préfère une autre approche.
D'abord, je recherche l'utilisateur à partir du login fourni.
dans un deuxième temps, je regarde si le login récupéré et celui fourni correspondent et si oui, alors je regarde si le mot de passe récupéré par la requette correspond à celui entré par l'utilisateur.

<?
class utilisateurs
{
    public $login;
    public $pass;
    public $error_string;
    private $connected;

    private $user_row;

    public function __construct()
    {
        $this->connected = FALSE;
    }

    private function get_user_row()
    {
        // recuperer les infos de l'utilisateur
$login=mysql_real_escape_string($this->login);
        $sql = 'SELECT login,password FROM login WHERE login=\'' . $login . '\'';
        $sql2=mysql_query($sql);
        $slq3=mysql_num_rows($sql2);
        $errno = mysql_errno($sql2);
        if(($errno == 0) AND ($sql3==1)){
                $this->user_row = mysql_fetch_assoc($sql2);
                return(TRUE);
        }else{
                $this->error_string = mysql_error();
                return(FALSE);
        }
    }
    public function test_login()
    {
        // vérifier si le login correspond
        if(isset($this->user_row['login']))
        {
         $login_to_test = $this->user_row['login'];
                if($login_to_test == $this->login)
                {
                 return(TRUE);
                }
                else
                {
                $this->error_string = 'Tentative d\'injection?';
                return(FALSE);
                }
        }
        else
        {
                $this->error_string = 'Login inconnu';
                return(FALSE);
        }
    }

    public function test_pass()
    {
       // vérifier si le mot de passe correspond
$pass_to_test = $this->user_row['password'];
        if($pass_to_test == $this->pass)
        {
         return(TRUE);
        }
        else
        {
$this->error_string = 'Mot de passe incorrecte';
return(FALSE);
        }
    }

    public function connect()
    {
        if(!$this->get_user_row())
        {
            $return(FALSE);
        }
        if(!$this->test_login())
        {
            $return(FALSE);
        }
        if(!$this->test_pass())
        {
            $return(FALSE);
        }
        $this->connected = TRUE;
    }

    public function is_connected()
    {
        return($this->connected == TRUE);
    }
}

$utilisateur = new utilisateurs();
$utilisateur->login = $login;
$utilisateur->pass = $pass;
if(!$utilisateur->connect())
{
    // pas de bol
    echo $this->error_string;
}
else
{
    echo 'connecté';
}
?>

Commentaire de Arto_8000 le 21/06/2010 18:58:38

masternico -> Le problème c'est que les injections SQL ne servent pas juste à contourner un login. Tu laisse carrément un accès à n'importe qui de faire des requêtes select. Tout dépendant de la base de données que tu utilises, ça laisse la possibilité d'avoir une fuite de ta base de données au complet.

Exemple d'entrée qui pose problème :

' UNION ALL SELECT * FROM INFORMATION_SCHEMA INTO OUTFILE 'chemin/Accessible/Du/Net.txt

Commentaire de Arto_8000 le 21/06/2010 19:28:31

ghuysmans99 -> mysql_query ne permet pas d'avoir plusieurs requêtes en même temps, ça va juste de sortir un exception si tu essaies de faire ça.