SYSTÈME D'ADMINISTRATION POUR CRÉATION D'ARTICLES

Commentaire de stay le 30/01/2013 09:19:50

Salut, tu t'es trompé. tu as mis, Niveau :Initié.
J'ai une question, en faite il y en aurait une dix-saine avant celle-ci du genre, c'est quoi un article pour toi.
Ma question, pour supprimer ou éditer un article, qu'est-ce-que tu as prévu ?

Commentaire de emilia123 le 31/01/2013 10:55:54

Bonjour,

Effectivement, cette source n'est malheureusement pas du niveau "initié".

Elle comporte des problèmes de sécurité et ne devrait donc pas être utilisée.
Problème SQL :
fichier systeme/script/envoi.php :
------------------------------------
$sql="INSERT INTO ".$bdd." (".$cols.") VALUES ('$titre','$text','$page') ";
------------------------------------
Aucune sécurisation n'est faite sur les variables ($titre, $text, $page) avant de les insérer dans la requête SQL.
En utilisant la bonne écriture, il est facile pour un attaquant de travailler directement sur ta base de donnée (insertion de donnée, création de table, suppression de données/tables, etc...).
Il FAUT TOUJOURS (majuscule, gras, souligné, couleur rouge) sécuriser les variables reçues d'un internaute en la passant par des fonctions du type :
mysql_real_escape_string(...) (dans le cadre de l'utilisation de mysql).

Problème fonctionnel :
fichier systeme/script/envoi.php :
------------------------------------
extract($_POST) ;
------------------------------------
il ne FAUT PAS (majuscule, gras, souligné, couleur rouge) utiliser ce type de fonction.
Dans ton script, si tu as une variable sensible (ex : gestion des droits d'accès) et que tu as oublié de l'initialiser, la présence de la fonction "extract" permet à un attaquant de l'initialiser à ta place... et donc de prendre le contrôle de ton script.
C'est à toi de savoir quelles sont les variables à récupérer et de les récupérer directement à la source ($_POST)...

Bonne continuation

EM.

Commentaire de Shad0white le 31/01/2013 18:57:14

Tout d'abord merci pour vos remarques très pertinente ! Je tient à préciser que vous pourriez surement trouver mon niveau de codage à la limite du ridicule et que j'ai sans doutes surestimer mes compétence à ce niveau. Je tient à dire aussi que ce projet a était réalisé dans le but d'aider certains/certaines débutant(e)s cherchant désespérément un système d'administration pour leurs articles (Évidemment, il y en a 10x mieux que le miens). Par la suite, je l'est aussi posté pour qu'on me critique et que j'essaie d'améliorer mon système notamment comme la sécurité qu'a évoquer emilia123. Je l'améliorait par la suite...

PS: STAY ! Je n'est pas compris ce que tu voulais dire au sujet de : "J'ai une question, en faite il y en aurait une dix-saine avant celle-ci du genre, c'est quoi un article pour toi." ??

Commentaire de Shad0white le 01/02/2013 21:21:10

Je te remercie d'avance pour ta réponse si vite apprécié ! Mon système, qui je dois le reconnaitre, n'a pas encore d'option supprimer ou éditer ! Je doit avouer que je ne me suis pas intéresser tout de suite à ce sujet même si mes connaissance débutants sont quelques avancés (peut-être pas dans l'art de la sécurisation !). Pour répondre à ta question, que tu attend avec impatience, pour moi un article est un groupement contenant ("un titre,une date d'édition,un contenu,...") comme l'on peut trouver sur un blog. Voilà, je n'est rien d'autres à dire à ce sujet. Pour en revenir à l'autre problème (Edition/Suppression) je m'y pencherait surement dans les jours à venir si le temps me le permet.

Cordialement.

Commentaire de Shad0white le 02/02/2013 10:08:13

Je comprend et je te remercie de ta patience ! Je vais suivre ton conseil ! Bonne continuation