Discussion :

[Halobox]

Bpnjour a tous, j'ai un problème, j'ai fait un script qui liste les fichiers a l'intérieur d'un dossier.

Le nom du dossier étant dans un $_GET,

En gros ca commence comme ca :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
$files = array(); 
$dir = $_GET['cate']; 
$folder = opendir($dir); 
while ($name = readdir($folder)) {

Et biensur quand on ne met pas de paramètre dans l'adresse, ca renvoie une erreur.

Le problème c'est que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(isset($_GET['cate']) == true){

ne fonctionne pas.


Quelqu'un a une idée ???


Merci d'avance

[julp]

Quelle est votre version de PHP ?

Il est inutile pour un test de tester si la valeur renvoyée est vrai, en l'occurence :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if (isset($_GET['cate'])) {

suffit. Votre code ne le montre pas mais il faut placer ce test avant l'affectation à $dir. Il vaudrait mieux alors écrire quelque chose comme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$dir = isset($_GET['cate']) ? $_GET['cate'] : '.';

Cordialement, Julp

[Halobox]

Edit, non ca ne fonctionne pas.


J'ai toujours ces 2 erreurs

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
Warning: readdir(): supplied argument is not a valid Directory resource in /var/www/free.fr/6/4/livecontent/index.php on line 43
 
Warning: closedir(): supplied argument is not a valid Directory resource in /var/www/free.fr/6/4/livecontent/index.php on line 51

Lorsque le tape http://monsite.index.php

Et par contre je n'ai aps d'erreur lorsque je tape http://monsite.index.php?

[trattos]

essaye alors de remplacer isset() par empty(), comme ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
if(!empty($_GET['cate']))
{
// Le chemin du dossier est bon
}

Par contre fais gafe au niveau de la sécurité car si on peut appeler un dossier contenant des fichiers distants de ton site ça peut être dangeureux!

[francis m]

faudrait peut-être que tu montres un peu plus de ton code, en l'occurence les lignes 40 à 55...

[Halobox]

Merci trattos ca fonctionne parfaitement

[vg33]

Par contre fais gafe au niveau de la sécurité car si on peut appeler un dossier contenant des fichiers distants de ton site ça peut être dangeureux!

+1
On ne répétera jamais assez qu'il ne faut jamais faire confiance aux données transmises par l'utilisateur, et donc qu'il faut systématiquement les valider avant de les utiliser dans un script.