PHP

[RÉSOLU]Problème de traitement d'un formulaire

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [RÉSOLU]Problème de traitement d'un formulaire

Buandflu

Bonjour !

Je suis en train de devenir fou avec mon site... J'essaye de supprimer des éléments dans ma base de données depuis un backoffice que je suis en train de coder.
Voici mon html :

Code :

<form action = "delete.php" method = "POST">
<div>Film :
<select id="film" name = "film">
<option value ="default">choisir le film à supprimer</option>
<?= getFilm() ?>
</select>
</div>
<div>Animation :
<select id="animation" name = "animation">
<option value ="default">choisir le dessin-animé à supprimer</option>
<?= getAnimation() ?>
</select>
</div>
<div>Série :
<select id="serie" name = "serie">
<option value ="default">choisir la série à supprimer</option>
<?= getSerie() ?>
</select>
</div>
<input type = 'submit' id = 'delete' name = 'delete' value = 'Supprimer' onclick = 'window.location.reload(true)'>
</form>

Voici mon php :

Code :

if (isset($_POST["delete"])) {
$pdo = dbconnect();
if ((isset($_POST['animation'])) or ( isset($_POST['serie']))) {
if (isset($_POST['animation']))
$id = $_POST['animation'];
else
$id = $_POST['serie'];
$sql = "delete from vid where id='$id'";
if ($pdo->exec($sql))
echo "Élement correctement supprimé de la base de données.";
else
echo "Il y a un problème";
$pdo = null;
}
} else {
$id = $_POST['film'];
$sql = "delete from film where id='$id'";
if ($pdo->exec($sql))
echo "Élement correctement supprimé de la base de données.";
else
echo "Il y a un problème";
$pdo = null;
}

Quelque chose ne tourne pas rond, mais je n'arrive pas à voir quoi. Le code peut supprimer seulement une animation. Le reste il n'y arrive pas...

Est-ce que quelqu'un pourrait me dépanner ?
Un grand merci d'avance !

Message édité par Buandflu le 10-06-2019 à 11:48:30

---------------

Publicité

Buandflu

Merci B4X pour ta réponse ! J'avais pas pensé à ça...
Je regarde ce soir pour corriger ça

Quand tu dis que mon code est vulnérable aux injections sql, comme c'est pas des input text, mais des selects (en gros le navigateur peut rien taper au clavier) c'est ok non ?

Message cité 1 fois

---------------

rat de combat

attention rongeur méchant!

Buandflu a écrit :

Quand tu dis que mon code est vulnérable aux injections sql, comme c'est pas des input text, mais des selects (en gros le navigateur peut rien taper au clavier) c'est ok non ?

Ah ben non, ça prend 3 lignes de script pour créer une requête qui va fou... en l'air ta base de données. Il faut TOUJOURS nettoyer ses données entrantes.
edit: Même aucune ligne de code avec les outils intégrés à Firefox!

Message édité par rat de combat le 07-06-2019 à 18:09:54

MaybeEijOrNot

but someone at least

Pour faire de l'injection SQL on n'utilise pas ta page HTML, on envoie directement une requête POST à ton fichier.php, à partir de là tu écries ce que tu veux dans ta requête POST afin de réécrire la requête SQL.
Néanmoins je vais mesurer les propos apportés, normalement, ton fichier n'est accessible que sur identification puisqu'il fait parti du backoffice, ce qui doit limiter les risques. Normalement le risque est présent si tu as différents niveaux d'autorisation dans le backoffice, à ce moment là, un utilisateur avec des droits faibles pourrait outrepasser ses autorisations avec une injection SQL.

Message cité 1 fois

---------------
C'est en écrivant n'importe quoi qu'on devient n'importe qui.

Buandflu

MaybeEijOrNot a écrit :

Néanmoins je vais mesurer les propos apportés, normalement, ton fichier n'est accessible que sur identification puisqu'il fait parti du backoffice, ce qui doit limiter les risques. Normalement le risque est présent si tu as différents niveaux d'autorisation dans le backoffice, à ce moment là, un utilisateur avec des droits faibles pourrait outrepasser ses autorisations avec une injection SQL.

Oui il faut bien sûr être connecté pour arriver sur delete.php.

Message édité par Buandflu le 09-06-2019 à 19:06:54

---------------

Buandflu

Effectivement. Du coup, comment on évite les injections ?

Message cité 1 fois

---------------

Buandflu

Buandflu a écrit :

Effectivement. Du coup, comment on évite les injections ?

Comme ~~google~~ lilo est mon ami, du coup j'ai mis des prepare statement (je sais pas comment on dit en Français). Ça donne ça :

Code :

if (isset($_POST["delete"])) {
$pdo = dbconnect();
if (!empty($_POST['animation'])) {
$sql = "delete from vid where id= :filmID";
$stmt = $pdo->prepare($sql);
$stmt->bindParam(':filmID', $_POST['animation'], PDO::PARAM_INT);
if ($stmt->execute())
echo "Élement correctement supprimé de la base de données.";
else
echo "Il y a un problème";
}elseif (!empty($_POST['film'])) {
$sql = "delete from vid where id= :filmID";
$stmt = $pdo->prepare($sql);
$stmt->bindParam(':filmID', $_POST['film'], PDO::PARAM_INT);
if ($stmt->execute())
echo "Élement correctement supprimé de la base de données.";
else
echo "Il y a un problème";
}elseif (!empty($_POST['serie'])) {
$sql = "delete from vid where id= :filmID";
$stmt = $pdo->prepare($sql);
$stmt->bindParam(':filmID', $_POST['serie'], PDO::PARAM_INT);
if ($stmt->execute())
echo "Élement correctement supprimé de la base de données.";
else
echo "Il y a un problème";
$pdo = null;
}
}

Ça marche bien. Vous voyez d'autre trucs pour éviter les injections les plus communes ?

---------------

Buandflu

Pas fou ça ! Merci B4X
J'ai une question qui me vient à l'esprit concernant les injections. Quelque chose comme ça est-il sûr ?
$stmt = $pdo->query("requête sql" )
J'arrive pas à trouver de réponse claire sur ~~google~~ lilo...

Message édité par Buandflu le 10-06-2019 à 11:43:53

---------------

rat de combat

attention rongeur méchant!

Une injection SQL n'est possible que quand on passe une variable non "nettoyée" dans une rêquete SQL. Si tu passes par le PDO normalement c'est bon, idem si tu fais une rêquete sans paramètres. Sans garantie, je ne parle pas PHP ni SQL.

FORUM HardWare.fr

Programmation

PHP

[RÉSOLU]Problème de traitement d'un formulaire

Sujets relatifs
Erreur code Formulaire de saisie excel	Problème de caractères spéciaux tkinter et fichier texte (Resolu)
Problème inclusion librairie arduino	[Python]Problème de split.
Problème pour créer une plateforme de conversion de monnaie avec vba	Problème requête GET serveurWeb/Microcontroleur
petit probleme avec grep	Création d'un "select" personnalisé
[VB/VBA/VBS] Problème script pour exécuter des programmes	Problème de map sur un array
Plus de sujets relatifs à : [RÉSOLU]Problème de traitement d'un formulaire

Page générée en 0.082 secondes