Discussion :

[larchicha]

Bonjour a tous,

Je m excuse a l avance, je suis sur un clavier US et je ne vois pas les accents donc pardon pour ma ponctuation desastreuse.

Comme le dit le titre, je bloque sur une erreur de segmentation dont voici le code :

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
24
25
26
27
28
void lire(int num){
// PROBLEME : CB Y A TIL DE FICHIERS DS LE DOSSIER AFIN DE COMPARER AVEC NUM --->  nbfiles()
// PB2 : Segmentation fault ?
	int cmp = nbfiles("home/larchicha/Desktop/recus");
	int i ; int descript ;
	char * num_str = malloc(50*sizeof(char)); 
	char * buff = malloc(100*sizeof(char)); 
	char * chemin = malloc(100*sizeof(char));;
	strcpy(chemin,"home/larchicha/Desktop/recus/");// chemin contient le debut du chemin moins le nom du fichier (1 2 3 4)
	for(i=0;i<cmp;i++){
		if (i == num){// tant qu on trouve pas le numero du symlink on boucle
			snprintf(num_str, 4, "%d", num);// convertit num : int -> string
			strcat(chemin,num_str);// concat le nom du fichier (1 ou 2 ou 3 ou 4) au chemin
 			descript = open(chemin,O_RDONLY);//on ouvre le fichier appeller 'num' 
			read(descript,buff,sizeof buff);// on lit le contenu du fichier
			printf("Le contenu du message est : %s\n",buff);//on l affiche
		} else {
		//perror("Le numero donne en argument ne correspond a aucun fichier : ");
		}
	}
}
 
int main(){
 
	nbfiles("recus");
	lire(2);
	return 0;
}

A savoir que la fonction nbfiles(const char *) prend le chemin d un repertoire et me renvoi le nombre de fichiers au il contient. (elle fonctionne, je l ai testee dans un autre fichier)

J ai bien entendu essaye de mettre en commentaire toutes les lignes qui me semblait potentiellement fausse mais je coince.

Merci pour l aide apportee.

[BlueMonkey]

Bonjour,

Je vois 2 problèmes potentiels :
- ligne 15 : sizeof buff, ne retourne pas la taille de la mémoire allouée par le malloc, mais l'équivalent de sizeof (char *).
(la taille de la mémoire alloué pour un pointeur).

-ligne 16 : printf (.."%s", buff); ici rien ne garanti que buff se termine pas le caractère de fin de chaîne \0 .
Par conséquent, il est tout à fait possible que printf cherche à accéder à de la mémoire non allouée.

Enfin je recommande, de :
- vérifier que la fonction open(..) ne retourne pas -1, indiquant un problème à l'ouverture de fichier.
- fermer les fichiers ouverts avec la fonction close()

[larchicha]

Merci de ta promptitude, je vais verifier de suite.

[larchicha]

J ai corrige mais l erreur reste toujours presente.
Etant donne que j ai besoin du print je l ai juste mis en commentaires et l erreur est restee.

Perso je pense plus a strcpy et strcat pour l erreur ( impression de deja vu).

Merci encore pour l aide apportee.

[BlueMonkey]

Comme ça je ne vois pas d'erreur sur la partie strcat ou strcpy.
Par contre vérifie que open() a fonctionné en testant descript soit différent de -1.
(parce qu'il n'y a jamais de close ... Et par conséquent il est possible que les fichiers soient restés ouvert, ce qui ferait échouer une nouvelle ouverture).

[Bktero]

Tu devrais rajouter des vérifications des allocations dynamiques.

[larchicha]

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
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <string.h>
#include <fcntl.h>
#include <dirent.h>
#define PATH "home/larchicha/Desktop/recus"
 
 
void lire(int num){
// PROBLEME : CB Y A TIL DE FICHIERS DS LE DOSSIER AFIN DE COMPARER AVEC NUM ___> FUCK YOU grace a nbfiles()
// PB2 : Segmentation fault ?
	int cmp = nbfiles("home/larchicha/Desktop/recus");
	int i ; int descript ;
	char * num_str = malloc(50*sizeof(char)); 
	if (num_str != NULL){
		char * chemin = malloc(100*sizeof(char));
		if (chemin != NULL){
			strcpy(chemin,"home/larchicha/Desktop/recus/");// chemin contient le debut du chemin moins le nom du fichier (1 2 3 4)
			for(i=0;i<cmp;i++){
				if (i == num){// tant qu on trouve pas le numero du symlink on boucle
					snprintf(num_str, 4, "%d", num);// convertit num : int -> string
					strcat(chemin,num_str);// concat le nom du fichier (1 ou 2 ou 3 ou 4) au chemin
		 			descript = open(chemin,O_RDONLY);//on ouvre le fichier appeller 'num' 
					if (descript != -1) {
						char * buff = ""; 
						read(descript,buff,100);// on lit le contenu du fichier
						//printf("Le contenu du message est : %s\n",buff);//on l affiche
					} else {
						perror("Erreur d ouverture du fichier : ");
					}
				} else {
					perror("Le numero donne en argument ne correspond a aucun fichier : ");
				}
 
			}
		}
	}
	close(descript);
 
}	
 
int main(){
 
	nbfiles("recus");
	lire(4);
	return 0;
}

Toujours a l execution : Segmentation Fault

[Bktero]

Code :

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

1
2
3
 
char * buff = "";
read(descript,buff,100);// on lit le contenu du fichier

Pourquoi le pauvre buff n'a pas le droit à un malloc ?

Sinon, une question comme ça... Pour qu'une allocation dynamique plante, y'a une chance sur 1000 nan ?

J'ai jamais eu de problème avec malloc mais je n'ai jamais fait de programmes vraiment gros avec des allocations vraiment grosses. Tu peux lire ça pour plus d'information.

[larchicha]

Ne m'en sortirais je donc jamais ?
Apres les corrections, toujours la meme erreur de segmentation...

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
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
void lire(int num){
// PROBLEME : CB Y A TIL DE FICHIERS DS LE DOSSIER AFIN DE COMPARER AVEC NUM ___> FUCK YOU grace a nbfiles()
// PB2 : Segmentation fault ?
	int cmp = nbfiles("home/larchicha/Desktop/recus");
	int i ; int descript ;
	char * num_str = malloc(50*sizeof(char)); 
	if (num_str != NULL){
		char * chemin = malloc(100*sizeof(char));
		if (chemin != NULL){
			strcpy(chemin,"home/larchicha/Desktop/recus/");// chemin contient le debut du chemin moins le nom du fichier (1 2 3 4)
			for(i=0;i<cmp;i++){
				if (i == num){// tant qu on trouve pas le numero du symlink on boucle
					snprintf(num_str, 4, "%d", num);// convertit num : int -> string
					strcat(chemin,num_str);// concat le nom du fichier (1 ou 2 ou 3 ou 4) au chemin
		 			descript = open(chemin,O_RDONLY);//on ouvre le fichier appeller 'num' 
					if (descript != -1) {
						char * buff = malloc(100*sizeof(char)); 
						 if (buff != NULL){
							read(descript,buff,100);// on lit le contenu du fichier
							printf("Le contenu du message est : %s\n",buff);//on l affiche
						}
						free(buff);
					} else {
						perror("Erreur d ouverture du fichier : ");
					}
 
				} else {
					perror("Le numero donne en argument ne correspond a aucun fichier : ");
				}
 
 
			}
		}
		free(chemin);
 
	}
	close(descript);
 
	free(num_str);
}
 
int main(){
 
	nbfiles("recus");
	lire(4);
	return 0;
}

Par ailleurs, le but de cette fonction me permet de lire un fichier dont le nom est "num" donc si vous pensez a un code plus simple, ne vous genez pas...
Merci du temps alloue a mon probleme