Existance d'un fichier, d'une apps.

**miron** · 06/08/2006, 16h52

Voila mon probleme est tout simple, je veux savoir si une application d'on je connaitrais l'@ existe.
J'ai essayer ceci, sans grand sucses...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
void main()
{
  char NOM_FICHIER[5][BUFSIZ],dossier_actuel[BUFSIZ]="\0",dossier_temp[BUFSIZ]="\0";
  FILE *P_FICHIER;      /* pointeur sur FILE */
  GetCurrentDirectory( 260 ,Dossier_actuel);
  sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");
  printf("%s ",dossier_temp);
  P_FICHIER = fopen(dossier_temp,"r");
  if ( P_FICHIER != NULL ) 
   { fclose(P_FICHIER);
      printf("1 : Boum.exe");
        //...   }
  //...
}

Mon "printf()"affiche le bon chemin, le fichier existe mais la condition n'est pas validée.
Une Idée ?

**David.Schris** · 06/08/2006, 17h20

Envoyé par miron

Une Idée ?

Deux :
- main() doit retourner un int (rien à voir avec ta question) ;
- ajouter un "#include <errno.h>" et un "extern int errno;" au début de ton code, puis faire un " perror("erreur"); " quand l'ouverture échoue (sans rien faire entre l'échec et l'appel à perror() !) et nous donner le message affiché par perror().

**miron** · 06/08/2006, 17h30

Envoyé par David.Schris

Deux :
- main() doit retourner un int (rien à voir avec ta question) ;

C'étais pour faire un programme pour vous qui soit compilable.
Moi depuis un certain temps je mets tj int.

- ajouter un "#include <errno.h>" et un "extern int errno;" au début de ton code, puis faire un " perror("erreur"); " quand l'ouverture échoue (sans rien faire entre l'échec et l'appel à perror() !) et nous donner le message affiché par perror().

Envoyé par perror()

Erreur: Invalid argument

**_kal_** · 06/08/2006, 17h35

A tout hasard, est-ce normal que dossier_actuel ait une majuscule à la ligne :

Code :

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

1
2
 
GetCurrentDirectory( 260 ,Dossier_actuel);

Le C est sensible à la casse

**miron** · 06/08/2006, 17h43

Envoyé par _kal_

A tout hasard, est-ce normal que dossier_actuel ait une majuscule à la ligne :

Code :

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

1
2
 
GetCurrentDirectory( 260 ,Dossier_actuel);

Le C est sensible à la casse

Je sais, et je pensse de plus que c'est fort possible que tu ait raison, je vais verifier.
Merci.
PS: Ca ne resout qd meme pas mon probleme!

VERIFICATION
Apres verif en fait non car Dossier_actuel est un pointeur sur dossier_actuel, donc
Dossier_actuel = @ de la premiere case du tableau dossier_actuel[].
Et
dossier_actuel = @ de la premiere case du tableau dossier_actuel[] aussi.
D'où cela ne change rien.

**_kal_** · 06/08/2006, 17h51

Envoyé par miron

Je sais, et je pensse de plus que c'est fort possible que tu ait raison, je vais verifier.
Merci.
PS: Ca ne resout qd meme pas mon probleme!

Je ne fait pas de programmation Windows, donc je ne connais pas la fonction GetCurrectDirectory. Est ce que celle-ci retourne une valeur ? Si oui, tu pourrai peut-être la tester avant d'ouvrir ton fichier.

**miron** · 06/08/2006, 17h52

Envoyé par _kal_

Je ne fait pas de programmation Windows, donc je ne connais pas la fonction GetCurrectDirectory. Est ce que celle-ci retourne une valeur ? Si oui, tu pourrai peut-être la tester avant d'ouvrir ton fichier.

Oui il me semble quelle retourne une valeur, mais peu importe puisquelle marche.
Comme je lai dit le chemin d'acces au fichier est le bon.

**David.Schris** · 06/08/2006, 17h57

Envoyé par miron

C'étais pour faire un programme pour vous qui soit compilable.

Mauvaise excuse : un main() retournant un int ne rend pas un programme non compilable (mais un main() retournant un int sans "return"...). Ce n'est pas pour t'embêter, c'est pour que tu prennes de bonnes habitudes.

Envoyé par miron

Envoyé par perror()

Erreur: Invalid argument

Envoyé par man errno

EINVAL Argument invalide.

Envoyé par man fopen

ERREURS
EINVAL Le mode fourni à fopen, fdopen, ou freopen était invalide.

Mmmh... Je reste perplexe...

Qu'affiche exactement (fais un copier/coller) ton " printf("%s ",dossier_temp); " ? (pense aussi à mettre la sortie de ton programme quand tu as un rpoblème, cela peut aider)

**_kal_** · 06/08/2006, 17h58

Est ce que tu initialises errno à 0 avant l'instruction :

Code :

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

1
2
 
P_FICHIER = fopen(dossier_temp,"r");

Personnellement, je ferai :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
int main()
{
  char NOM_FICHIER[5][BUFSIZ],dossier_actuel[BUFSIZ]="\0",dossier_temp[BUFSIZ]="\0";
  FILE *P_FICHIER;      /* pointeur sur FILE */
  GetCurrentDirectory( 260 ,Dossier_actuel);
  sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");
  printf("%s ",dossier_temp);
  errno = 0;
  P_FICHIER = fopen(dossier_temp,"r");
  if (P_FICHIER == NULL)
  {
     perror(dossier_temp);
     exit(EXIT_FAILURE);
  }
  if ( P_FICHIER != NULL ) 
   { fclose(P_FICHIER);
      printf("1 : Boum.exe");
        //...   }
  //...
 
  return EXIT_SUCCESS;
}

**miron** · 06/08/2006, 18h02

Envoyé par David.Schris

Mauvaise excuse : un main() retournant un int ne rend pas un programme non compilable (mais un main() retournant un int sans "return"...). Ce n'est pas pour t'embêter, c'est pour que tu prennes de bonnes habitudes.
Mmmh... Je reste perplexe...
Qu'affiche exactement (fais un copier/coller) ton " printf("%s ",dossier_temp); " ? (pense aussi à mettre la sortie de ton programme quand tu as un rpoblème, cela peut aider)

Ces dernieres lignes mon fait comme un electrochoc, et j'ai penssasi dou pouvais provenir le probleme !!
Ce n'est pas

Code :

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

sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

Mais

Code :

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

sprintf(dossier_temp,"%s%s",dossier_actuel,"\\Boum\\Boum.exe");

Quil fallais mettre!!!
Et la le code d'erreur est le bon:

No such file or directory

(car pour essais je l'ai suprimé)
Donc un grand

**_kal_** · 06/08/2006, 18h05

Il y a un truc aussi qui m'intrigues, c'est l'utilisation de sprintf avec dossier_temp qui est de taille BUFSIZE :

Code :

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

1
2
 
sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

dossier_temp est un tableau de char de même taille que dossier_actuel. Or, tu va stocker dans ce tableau une chaine dont on ne connais pas la taille. Utilise plutot snprintf, qui est plus sur :

Code :

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

1
2
 
snprintf(dossier_temp, sizeof dossier_temp, "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

**miron** · 06/08/2006, 18h09

Envoyé par _kal_

Il y a un truc aussi qui m'intrigues, c'est l'utilisation de sprintf avec dossier_temp qui est de taille BUFSIZE :

Code :

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

1
2
 
sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

dossier_temp est un tableau de char de même taille que dossier_actuel. Or, tu va stocker dans ce tableau une chaine dont on ne connais pas la taille. Utilise plutot snprintf, qui est plus sur :

Code :

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

1
2
 
sprintf(dossier_temp, sizeof dossier_temp, "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

sprintf ou snprintf.???

**_kal_** · 06/08/2006, 18h12

Envoyé par miron

sprintf ou snprintf.???

snprintf pardon, le 'n' a sauté au passage. D'ailleurs, ca m'étonne que ton programme ait reussi l'instruction du sprintf sans broncher.

Aussi, ici j'ai utilisé sizeof dossier_temp dans l'expression avec snprintf, car ton tableau est de type char. Tu peux tres bien mettre :

Code :

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

1
2
 
snprintf(dossier_temp, BUFSIZ, "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

Ou encore :

Code :

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

1
2
 
snprintf(dossier_temp, sizeof dossier_temp / sizeof dossier_temp[0], "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

Si ton tableau avait été d'un autre type que char, il n'aurai surtout pas fallu utiliser sizeof(dossier_temp) tout seul.

EDIT : suppression du E à BUFSIZE, ça risquerai d'en troubler quelque uns

**David.Schris** · 06/08/2006, 18h14

Envoyé par miron

sprintf ou snprintf.???

snprintf()

**miron** · 06/08/2006, 18h39

Envoyé par _kal_

Code :

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

1
2
 
snprintf(dossier_temp, BUFSIZE, "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

Pourquoi BUFSIZE et pas BUFSIZ

**_kal_** · 06/08/2006, 18h41

Envoyé par miron

Pourquoi BUFSIZE et pas BUFSIZ

Effectivement, BUFSIZ. D'ailleurs, c'est une macro que tu définie toi même ?

**miron** · 06/08/2006, 19h01

Envoyé par _kal_

Effectivement, BUFSIZ. D'ailleurs, c'est une macro que tu définie toi même ?

Euuu
Arrette tu va trouver une faille dans mon programme qui fonctionne tres bien d'ailleur...

**_kal_** · 06/08/2006, 19h09

Envoyé par miron

Euuu
Arrette tu va trouver une faille dans mon programme qui fonctionne tres bien d'ailleur...

C'est mon objectif. Un programme peut sembler bien fonctionner à première vue, et disposer de trou de sécurité dangereux.
Un conseil : evite d'utiliser les fonctions de copie qui n'établissent aucun contrôle sur le nombre d'élement copié.

Exemple :

sprintf -> snprintf
strcpy -> strncpy
gets -> fgets

Cela t'evitera des mals de têtes à l'avenir

**miron** · 06/08/2006, 20h52

Envoyé par _kal_

C'est mon objectif. Un programme peut sembler bien fonctionner à première vue, et disposer de trou de sécurité dangereux.
Un conseil : evite d'utiliser les fonctions de copie qui n'établissent aucun contrôle sur le nombre d'élement copié.

Exemple :