par
jpaul » 06 mai 2007, 12:58
Bonjour,
Moi aussi, je faisais cela avec une simple boucle foreach() avant que l'on attire mon attention sur le fait que cela risquait de poser des problèmes de sécurité.
En effet une telle boucle permet à une personne mal intentionnée de forcer le passage d'une variable non prévue.
Cela peut être par exemple (avec
get mais c'est tout aussi facile pour
post) :
mon_ulr?Admin=1
Et pouf, avec un peu de chance, l'apprentit pirate à réussit : il est en mode admin...
J'en ai conclu que c'est une très mauvaise habitude, car même si les développements que l'on réalise aujourd'hui ne sont pas sensibles, qui sait si un jour...
Et une mauvaise habitude est toujours difficile à perdre...
Il faut toujours contrôler et valider toutes les variables passées.