GESTIONNAIRE DE NEWS BASIQUE AVEC UPLOAD D'IMAGES
Morphinof
Messages postés
255
Date d'inscription
vendredi 20 avril 2007
Statut
Membre
Dernière intervention
9 août 2013
-
5 avril 2011 à 09:58
brabri Messages postés 1 Date d'inscription lundi 17 mars 2008 Statut Membre Dernière intervention 25 janvier 2012 - 25 janv. 2012 à 14:36
brabri Messages postés 1 Date d'inscription lundi 17 mars 2008 Statut Membre Dernière intervention 25 janvier 2012 - 25 janv. 2012 à 14:36
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/53032-gestionnaire-de-news-basique-avec-upload-d-images
https://codes-sources.commentcamarche.net/source/53032-gestionnaire-de-news-basique-avec-upload-d-images
25 janv. 2012 à 14:36
Je crois que j'ai trouvé une faille xss dans le "titre" ?
Merci bien en tout cas.
>
...
<?php echo htmlspecialchars($_POST['titre']); ?>
...
non ? Qu'en pensez-vous ? est-ce suffisant ?
11 avril 2011 à 09:36
merci les gars !
MDR !
5 avril 2011 à 11:34
je vais blinder la fonction plus tard
avec d'autres astuces dont le mine
faudra juste ajouter des else if ou un switch
mais là elle déjà basiquement sécurisante
j'ai fait au plus vite
merci pour les conseils
a++
5 avril 2011 à 11:26
Par contre tu teste pas le type mime, et niveau gestion d'erreur on peux pas savoir ou la fonction a retourne false.
Tu devrais jeter une exception :
throw new Exception(__FUNCTION__.'::FileNameFault'); et hrow new Exception(__FUNCTION__.'::FileExtentionFault'); par exemple
5 avril 2011 à 11:15
j'ai fait une mise à jour pour l'upload
avec une fonction qui te rappelera quelque chose
je vais la poster de suite ...
le reste suivra dans les jours à venir
a++
5 avril 2011 à 10:51
5 avril 2011 à 10:32
Tu as completement raison c'est pas securisé du tout et pas beau, mais bon, comme je l'ai dit c'est un bout de code qui vient du forum je vais ameliorer ça quand j'aurai 5 minutes
c'est un canevas ...
1/ je vais ajouter une table membres avec statut admin
2/ securise upload (en premier)
3/ peut etre une miniaturisation
4/ un editeur javascript à la place du textarea
5/ un install.php
6/ MVC oui aussi je vais separer
...
a++
5 avril 2011 à 10:09
5 avril 2011 à 10:05
5 avril 2011 à 09:58
http://www.phpcs.com/codes/UPLOADEUR-FICHIERS-MULTIPLES-V1_52956.aspx