Discussion :

[pazze]

Bonjour,

Mon problème est le suivant, j'ai un fichier qui contient le résultat d'un ls et un char * qui contient un chemin d'accès au dossier.

Je dois ouvrir chaque fichier contenu dans le résultat de ls. Pour ce faire, je dois concaténer le chemin d'accès(/home/user...) + le nom du fichier (foo.txt).

Je le fais de cette façon

Code :

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

1
2
3
4
5
6
7
8
9
 
const char *path = "/home/user/logs";
 
void envoiFichier(char *nomFichier) {
	char *fichier;
	int longueur = 256;
	fichier = malloc(longueur);
	memset(fichier, '\0', sizeof(fichier));
       snprintf(fichier, longueur, "%s/%s\0", path, nomFichier);

Puis j'imprime le résultat, ça me donne quelque chose de correct (/home/user/foo.txt ) . Mais lorsque je veux passer le résultat de la concaténation à fopen, j'obtiens un No Such file or directory alors que le fichier existe bel et bien.

Code :

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

if ((file = fopen(fichier, "r")) == NULL) perror("Erreur dans l'open du fichier.info");

Je crois que c'est un problème de '\0' ou quelque chose comme ça, mais j'ai aucune idée de comment le résoudre !
Pouvez-vous m'aider ?

[jeroman]

Salut

Pour ton problème, je ne saurai pas t'aider. Par contre, si tu affiches la chaine et que ça correspond au résultat attendu, le problème vient forcément d'ailleurs. Problème de permission ?

Par contre, il y a moyen d'alléger ton code. Pourquoi utiliser malloc et memset, qui sont inutiles ici ?

Voilà plus simple :

Code :

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

1
2
3
4
5
6
7
8
9
void envoiFichier(char const * const nomFichier)
{
	char fichier[256];
 
	snprintf(fichier, 256, "%s/%s", path, nomFichier);
 
/*	(...) */
 
}

Si tu tiens absolument à utiliser le pointeur path comme objet global, le mieux est de le rendre constant, afin d'éviter les mauvaises surprises :

Code :

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

const char * const path = "/home/user/logs";

[pazze]

Merci, je vais essayer de retirer les malloc etc...

Je dispose des bonnes permissions, et si j'écris directement le chemin d'accès dans le fopen(), je n'ai pas d'erreur ! C'est pour ça que je ne comprend pas, il affiche le bon résultat

/home/user/logs/foo.txt (= fichier)

mais dans le fopen() :

fopen(fichier) != fopen("/home/user/logs/foo.txt")

D'où ça pourrait venir ?

[jeroman]

si j'écris directement le chemin d'accès dans le fopen(), je n'ai pas d'erreur !

C'est pas normal.

Montre ta fonction, au complet.

[pazze]

C'est une application client-serveur qui liste le contenu d'un dossier, et envoie le tout au serveur.

La partie qui pose problème se trouve dans le client.


Voila la fonction avec le fopen() qui pose problème

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
void envoiFichier(int socket, char *nomFichier) {
	FILE* file;
	char *fichier;
	int longueur = 256;
        char buffer[4096],*message;
	fichier = malloc(longueur);
	memset(fichier, '\0', sizeof(fichier));
       snprintf(fichier, longueur, "%s/%s\0", path, nomFichier);
       printf(fichier);
	if ((file = fopen(fichier, "r")) == NULL) perror("Erreur dans l'open du fichier.info"); //fonctionne avec youpi
	else {
		message = malloc(4096);
		memset(message, '\0', sizeof(message));
		while (fgets(buffer,4096,file) !=  NULL) {
			strcat(message,buffer);	
		}
		envoyer(socket, message);
		fclose(file);
	}
}

La fonction qui appelle envoiFichier

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 traitement(int socket) {
	FILE* file;
        char *buffer;
	if ((file = fopen("output.txt", "r")) == NULL) perror("Erreur dans l'open de output.txt");
 
 
	buffer = malloc(4096);
	memset(buffer,'\0',sizeof(buffer));
 
	while (fgets(buffer,4096,file) !=  NULL) {
		envoyer(socket, buffer);
		envoiFichier(socket,buffer);
	}
	fclose(file);
}

[droggo]

Sia,

Je ne vois pas trop où est ton problème, mais tes memset ne font pas ce que tu penses : sizeof(fichier) renvoie la taille de la variable fichier, donc la taille d'un pointeur, et pas la taille réservée pour ce pointeur !!

[gl]

Puis j'imprime le résultat, ça me donne quelque chose de correct (/home/user/foo.txt ) .

Plutôt qu'un simple printf(fichier), utilises une impression du style printf("<%s>\n", fichier) et donne le résultat exact (c'est à dire y compris les retours à ligne) obtenu, voire affiche le code hexa de chaque caractère jusqu'au \0 terminal.

[gangsoleil]

Bonjour,

Tu testes le code de retour de fopen, mais pas celui de malloc. Pourquoi ?
Tu ne liberes pas la memoire allouee.
Le fait de travailler en allocation dynamique ne sert a rien : tu n'as rien de dynamique dans ton code, tu travailles sur une chaine de 256 char.
Et enfin, snprintf te garantit que ta chaine se termine par un \0. Il n'est donc pas utile de l'ecrire toi meme.

[matafan]

Tu as très probablement un '\n' à la fin de ton nom de fichier. Il faut l'enlever (ie. mettre à '\0' à la place) avant de passer la chaine à open, puisque le nom de fichier sur ton disque ne contient, lui, pas de newline.

[pazze]

le résultat de printf("<%s>\n", fichier) donne


</home/user/logs/Server.info
>


EDIT : j'ai réglé le problème avec un strtok() Merci

[matafan]

Mais non malheureux

Il suffit de regarder la chaine à l'indice retourné par snprintf(), et si c'est un '\n', y écrire un '\0'.

[pazze]

J'ai essayé, mais j'obtiens des segfault sans trop comprendre pourquoi et j'ai du mal à comprendre comment ça marche.

Par exemple, index = snprintf....

à [index], j'ai bien \0
à [index+1], j'ai \n
et à [index-1], j'ai ni l'un ni l'autre

Dans ce cas la, comme le \0 est avant le \n, ça ne pose pas de problème je pense....

[matafan]

C'est à index - 1 en fait. A condition que tu vires ce "\0" qui ne sert à rien dans ton snprintf().