Discussion :

[danathane]

Bonjour, j'aimerais savoir une chose,
dans mon main(), j'ouvre un fichier, et je passe le pointeur à une fonction pour qu'elle écrive dans le fichier. Si je fais ca c'est parce que la fonction qui ouvre le fichier et très souvent appelé, donc j'économise du temps machine.
le problème c'est que quand j'essai d'écrire dans le fichier il me dit "access violation"

voilà le code :

ca c'est dans le main. et c'est la fonction show packet à laquelle je passe le pointeur.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
FILE *fp=fopen("./fichier/resultatfonction.txt","w");
 
desc = pcap_open_offline("./fichier/fichiertest.pcap", errbuf); 
if (desc == NULL)
	printf("pcap_open_offline: %s\n", errbuf), exit(2);
 
 
 
while ((packet = pcap_next(desc, &hdr)))
	{
	showpacket(hdr, packet,fp);
	}

la fonction showpacket :

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
 
 
void showpacket(struct pcap_pkthdr hdr, const u_char *packet, FILE *fp)
{
char  chaineretour[10000];     
struct tm *mytime;
int i, j, k;
 
mytime = localtime(&hdr.ts.tv_sec);
 
sprintf(chaineretour, "%.2d:%.2d:%.2d.%.6i", mytime->tm_hour, mytime->tm_min, mytime->tm_sec,
			(int)hdr.ts.tv_usec);
 
fwrite(chaineretour,strlen(chaineretour),sizeof(chaineretour),fp);
 
printf(", Len: %i (0x%x)\n",hdr.caplen,hdr.caplen);

Je pense que l'erreur vient des pointeur mais j'ai un peu de mal à débrouiller tout ca.
merci pour le coup de main

[Thierry Chappuis]

Tu ne vérifies pas si l'ouverture du fichier a échoué (i.e. fp reçoit de fopen() une valeur différente de NULL)?

Thierry

[danathane]

Non le fichier s('ouvre bien ya pas de soucis la dessus, j'ai testé la ligne avant de faire un jolie copier,coller. Le problème doit venir du pointuer que je passe en argument. je sais plus si dans l'appel de la fonction je dois mettre &fp ou juste fp, et dans la fonction si je dois mettre FILE *fp, ou FILE **fp.

[Thierry Chappuis]

Non le fichier s('ouvre bien ya pas de soucis la dessus, j'ai testé la ligne avant de faire un jolie copier,coller. Le problème doit venir du pointuer que je passe en argument. je sais plus si dans l'appel de la fonction je dois mettre &fp ou juste fp, et dans la fonction si je dois mettre FILE *fp, ou FILE **fp.

Si fp est de type pointeur sur FILE et que ta fonction attend un argument de type FILE *, il n'y a pas de raison de passer autre chose que fp en paramètre. A part si la fonction a besoin de modifier la valeur du pointeur lui même, je ne vois aucune raison d'utiliser un paramètre de type FILE ** dans l'interface de ta fonction.

Thierry

[danathane]

Bon après avoir tester l'ensemble des possibilté, voilà comment il faut faire pour passer le pointeur d'un fichier à une fonction pour qu'elle puisse écrire dedans :

dans le main :

Code :

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

showpacket(machin,&fp);

et l'en-tete de la fonction :

Code :

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

1
2
3
4
5
6
7
 
void showpacket(int machin, FILE ** fp){
//pour l'utiliser :
 
fputs("truc",*fp);
 
}

grâce à cela je peux modifier mon fichier sans avoir besoin de l'ouvrir à chaque appel de la fonction.

Merci pour les conseils

[Thierry Chappuis]

Bon après avoir tester l'ensemble des possibilté, voilà comment il faut faire pour passer le pointeur d'un fichier à une fonction pour qu'elle puisse écrire dedans :

dans le main :

Code :

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

showpacket(machin,&fp);

et l'en-tete de la fonction :

Code :

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

1
2
3
4
5
6
7
 
void showpacket(int machin, FILE ** fp){
//pour l'utiliser :
 
fputs("truc",*fp);
 
}

grâce à cela je peux modifier mon fichier sans avoir besoin de l'ouvrir à chaque appel de la fonction.

Merci pour les conseils

Si tu veux juste que ta fonction écrive dans le fichier sans besoin de modifier la valeur du pointeur lui-même, je ne vois pas l'utilité de passer par un argument de type FILE **.

Thierry

[Sve@r]

Bon après avoir tester l'ensemble des possibilté, voilà comment il faut faire pour passer le pointeur d'un fichier à une fonction pour qu'elle puisse écrire dedans :

dans le main :

Code :

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

showpacket(machin,&fp);

et l'en-tete de la fonction :

Code :

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

1
2
3
4
5
6
7
 
void showpacket(int machin, FILE ** fp){
//pour l'utiliser :
 
fputs("truc",*fp);
 
}

grâce à cela je peux modifier mon fichier sans avoir besoin de l'ouvrir à chaque appel de la fonction.

Si la seule façon pour une fonction de modifier un fichier est de recevoir un FILE **, alors pourquoi fputs (qui a quand-même une action de modification incontournable) reçoit seulement un FILE * ???
Ton problème doit se situer ailleurs...

[Emmanuel Delahaye]

Bon après avoir tester l'ensemble des possibilté, voilà comment il faut faire pour passer le pointeur d'un fichier à une fonction pour qu'elle puisse écrire dedans :

dans le main :

Code :

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

showpacket(machin,&fp);

et l'en-tete de la fonction :

Code :

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

1
2
3
4
5
6
7
 
void showpacket(int machin, FILE ** fp){
//pour l'utiliser :
 
fputs("truc",*fp);
 
}

C'est une complication inutile. Ceci

Code :

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

showpacket(machin, fp);

Code :

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

1
2
3
4
5
6
 
void showpacket(int machin, FILE * fp){
   //pour l'utiliser :
   fputs("truc", fp);
 
}

doit fonctionner. Ou alors tu ne montres pas tout.

Le fclose(), tu le fais quand ?