broncowdd/googol — pull request #23 by JeromeJ (2013-09-10 13:14:27+00:00)
Title: XSS : Une bonne fois pour toutes ?

- Correction de la faille XSS liée à $_GET['mod']
- Correction de la faille XSS liée à $_GET['start']
- Correction de la double faille XSS liée à $_GET['couleur'] et $_GET['taille']

Double car si PHP affiche (toutes) les erreurs, un Notice index error est émis si $_GET['couleur'] a une valeur invalide, qui est elle ensuite affichée à l'écran sans être protégée par PHP (bah oui ils sont cons que vous voulez-vous http://www.olissea.com/mb/links/1/?CyDcGA ) (J'ai pas corrigé ça (la gestion de mauvaise valeur pour éviter l'index error) mais au moins ça désactive la faille XSS supplémentaire)

- Correction de la faille XSS liée à $_GET['lang']

Tu sanitizes des données parfois ? :p

Bronco a beau nous faire des programmes géniaux (super trop cools même !) faudrait plus le laisser publier un code sans que quelqu'un review complètement sa sécurité :p (j'ai pas revu le code en entier perso)
Ton ratio de faille XSS / code est trop gros l'ami ^^

- Ensuite, je n'ai pas compris l'intérêt de la ligne 307,
$q_txt = htmlspecialchars($_GET['q']); n'aurait-il pas suffit ?
Ton code porte à confusion:
if (!$q_txt=htmlspecialchars($_GET['q'])){$q_txt=$_GET['q'];}
Quand est-ce que htmlspecialchars($_GET['q']) serait vide, et si c'est le cas, en quoi serait-il différent de $_GET['q'] sans htmlspecialchars ?