Erreur de segmentation : à cause d'un strcmp()

**titou624** · 22/12/2008, 16h31

Tout est dit dans le titre à la compilation aucun problème mais lors de l'exécution de mon programme.
Je vous met le code et les explications. Ce code à pour but de chercher les fichiers comportant l'extension passée en paramètre puis de modifier le contenu en le cryptant en rot13. Le code n'est pas encore finni, pour l'instant je veux juste qu'il m'affiche les fichiers trouvés. Il m'affiche bien les fichiers mais ensuite il m'affiche "erreur de segmentation".
C'est un sujet de TP donc ce code n'a pas vraiment d'utilité

...
Merci d'avance pour l'aide que vous pourrez m'apporter

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
 
/* 
crypte rep_depart extension nbMaxFils (Nb maximum de fils qui crypteront chacun des fichiers trouvés)
 
tube :
	-> le père ouvre les chemin du fichier a traiter 
	-> les fils signalent qu'ils ont terminé
 
Tous les fichiers possédant cette extension effectue un cryptage rot13
	-> par un processus fils
	-> communication par tube
*/
 
#include <iostream>
#include <stdlib.h>
#include <sys/stat.h>
#include <sys/types.h>
#include <unistd.h>
#include <stdio.h>
#include <dirent.h>
#include <string.h>
#define MAXNAME 100
 
using namespace std;
/*
	on créé le tube en premier
*/
int p[2];
 
// Fonction de cryptage //
 
char rot13(char n){
	if(n<'z' && n>'z'){
		n += 'a';
		n = (n+13)%26;
	}
	else if(n<'Z' && n>'A'){
		n += 'A';
		n = (n+13)%26;
	}
	return n;		
}
 
 
int main(int argc, char **argv){
// Test des arguments
	if(argc!=4){
		cout << "Usage: crypte rep_depart ext nbProc" << endl;
		exit(2);
	}
//ouverture du tube
	pipe(p);
 
// Déclaration des variables
	int nbProc;
	struct stat buf;
	struct dirent *ent;
	char nom[MAXNAME];
	char *ext;
	char *nomRep;
	DIR *rep;
 
// On transtype le nombre de processus
	nbProc = atoi(argv[3]);
	ext = argv[2];
	nomRep = argv[1];
 
//ouverture du dossier
	rep = opendir(nomRep);
 
//Test si le dossier s'ouvre correctement
	if(!rep){	
// On signale l'erreur
		perror("opendir");
	}
 
// Tant qu'il ya des fichiers
	while(ent=readdir(rep))
	{
 
// On construit l'adresse du fichier
			strcpy(nom,nomRep);
 
// Si le fichier est différent de ./ ou ../
			if(strcmp(ent->d_name,".") && strcmp(ent->d_name,"..")){
				strcat(nom,"/");
				strcat(nom,ent->d_name);
 
// On récupère les informations sur le fichier
// lstat permet d'éviter de compter les liens symbolique
				if(lstat(nom,&buf)!=-1){
// Si c'est un fichier
						if(S_ISREG(buf.st_mode)){							
								char *tmp = strrchr(ent->d_name,(int)'.');
								tmp++;
							if(strcmp(tmp,ext)==0){
								cout << nom << endl;
/*
 // On écrit le chemin dans le tube
								close(p[0]);
								write(p[1],&nom,strlength(nom)*sizeof(char));
								close(p[1]);
*/
							}		
						}
				}	
 
			}
	}
// On ferme le repertoire
	closedir(rep);
 
}

**koala01** · 22/12/2008, 16h51

Salut,

Ce code ressemble furieusement à du C pur, y compris dans les fichiers d'en-tête inclus.

Avant de le déplacer dans la rubrique C, je voudrais confirmation du fait que ton objectif est de programmer en C, et non en C++...

Pourrais tu me donner cette confirmation

**ram-0000** · 22/12/2008, 17h01

heu ...

Code :

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

1
2
3
#include <iostream>
using namespace std;
cout << "Usage: crypte rep_depart ext nbProc" << endl;

c'est pas du C mais bien du C++

**koala01** · 22/12/2008, 17h10

Envoyé par ram-0000

heu ...

Code :

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

1
2
3
#include <iostream>
using namespace std;
cout << "Usage: crypte rep_depart ext nbProc" << endl;

c'est pas du C mais bien du C++

Ah, j'ai zappé ce qui est sans doute la seule ligne qui soit effectivement en C++ (il est vrai que j'ai parcouru le code en diagonale

)

**titou624** · 22/12/2008, 17h13

Oui effectivement au départ la prof voulais qu'on réalise le code en C mais par soucis de simplicité je préfère les cout <<

Une idée pour l'erreur ?

**3DArchi** · 22/12/2008, 17h27

Il manque la définition de ta structure struct dirent et ta fonction readdir.
Comme c'est du C plutôt que tu C++, je déplace le sujet vers le forum C où les réponses seront plus appropriées.

D'avance, merci aux intervenant du forum C de ne pas s'outrer de ce mélange incongru et de ne se concentrer que sur la partie problématique en C.

**titou624** · 22/12/2008, 17h38

Pour ne choquer personne j'ai "traduit le code en C. cela sera peut-être plus clair pour tout le monde ici

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
/* 
/*
crypte rep_depart extension nbMaxFils (Nb maximum de fils qui cypteront chacun des fichiers trouvés)
 
tube :
	-> le père ouvre les chemin du fichier a traiter
	-> les fils signalent qu'ils ont terminé
 
Tous les fichiers possédant cette extension effectue un cryptage rot13
	-> par un processus fils
	-> communication par tube
*/
 
 
#include <sys/stat.h>
#include <sys/types.h>
#include <stdlib.h>
#include <stdio.h>
#include <dirent.h>
#include <string.h>
#define MAXNAME 100
 
/*
	on créé le tube en premier
*/
int p[2];
 
/* Fonction de cryptage // */
 
char rot13(char n){
	if(n<'z' && n>'z'){
		n += 'a';
		n = (n+13)%26;
	}
	else if(n<'Z' && n>'A'){
		n += 'A';
		n = (n+13)%26;
	}
	return n;
}
 
 
int main(int argc, char **argv){
/* Test des arguments */
	if(argc!=4){
		printf("Usage: crypte rep_depart ext nbProc \n");
		exit(2);
	}
/* ouverture du tube */
	pipe(p);
 
/* Déclaration des variables */
	int nbProc;
	struct stat buf;
	struct dirent *ent;
	char nom[MAXNAME];
	char *ext;
	char *nomRep;
	DIR *rep;
 
/* On transtype le nombre de processus */
	nbProc = atoi(argv[3]);
	ext = argv[2];
	nomRep = argv[1];
 
/* ouverture du dossier */
	rep = opendir(nomRep);
 
/* Test si le dossier s'ouvre correctement */
	if(!rep){
/* On signale l'erreur */
		perror("opendir");
	}
 
/* Tant qu'il ya des fichiers */
	while(ent=readdir(rep))
	{
 
/* On construit l'adresse du fichier */
			strcpy(nom,nomRep);
 
/* Si le fichier est différent de ./ ou ../ */
			if(strcmp(ent->d_name,".") && strcmp(ent->d_name,"..")){
				strcat(nom,"/");
				strcat(nom,ent->d_name);
 
/* On récupère les informations sur le fichier */
/* lstat permet d'éviter de compter les liens symbolique */
				if(lstat(nom,&buf)!=-1){
/* Si c'est un fichier */
						if(S_ISREG(buf.st_mode)){
								char *tmp = strrchr(ent->d_name,(int)'.');
								tmp++;
							if(strcmp(tmp,ext)==0){
								printf("%s \n",nom);
/*
   On écrit le chemin dans le tube
								close(p[0]);
								write(p[1],&nom,strlength(nom)*sizeof(char));
								close(p[1]);
*/
							}
						}
				}
 
			}
	}
/* On ferme le repertoire */
	closedir(rep);
 
}

**titou624** · 22/12/2008, 22h33

Est-ce que quelqu'un sait comment un strcmp() peut causer une erreur de segmentation ?
merci à tous parce que ça fait un long moment que je suis bloqué

**ram-0000** · 22/12/2008, 22h40

Envoyé par titou624

Est-ce que quelqu'un sait comment un strcmp() peut causer une erreur de segmentation ?

strcmp() compare 2 chaines de caractères dont les pointeurs sont passés en paramètres à la fonction.

Je suppose qu'il suffit qu'un des deux pointeurs soit malade pour que cela parte en live. Quelques idées :

Pointeur NULL
Pointeur fou (pointe n'importe où dans la mémoire)
chaine de caractère pas terminée par 0

**titou624** · 22/12/2008, 22h52

Tout d'abord merci pour ton aide !

J'ai ajouté ce code pour tester les différentes erreurs possible mais rien n'y fait ...

Code :

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

1
2
3
4
 
if(tmp != NULL && tmp[strlen(tmp)]=='\0'){
	if(strcmp(tmp,ext)==0)
		printf("%s \n",nom);

toujours cette même erreur mais le plus navrant c'est que le code s'exécute correctement et l'erreur vient à la fin....

**ram-0000** · 22/12/2008, 23h01

attention, si ton pointeur est fou ou que la chaine n'est pas terminée par 0, l'appel à strlen() va lui aussi partir en live.

Je ne connais pas de manière simple de tester si un pointeur est fou ou si la chaine est bien terminée.

Soit tu es rigoureux dans ta manière d'écrire et tu supprimeras 95% de ces problèmes (les 5 % restant, c'est que tu n'es pas assez rigoureux

) ou alors tu peux utiliser des programmes (purify mais c'est est payant, je ne sais pas si Valgrind sous Linux sait faire la même chose) qui permettent de valider tes paramètres.

Après, il te reste le meilleur ami du développeur, c'est à dire le debugger

**titou624** · 22/12/2008, 23h05

merci beaucoup je ne connaissais pas cet outil je vais donc tester valgrind

Bonne soirée à toi

**lilington** · 23/12/2008, 08h28

d'abort moi j'aime pas les wildes pointers. si tu pouvais prendre la bonne habitude d'orienter tes pointers a la declaration vers NULL ce serait mieux que des des pointeurs qui pointent vers ou bon leur semble.
ensuite quand tu travailles avec des chaines de caracteres prefere utiliser string.h ou alors fais des copies octets par octets car c'est possible que ton

Code :

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

 ext=argv[2]

soit un probleme je te conseille donc de tester avec :

Code :

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

1
2
3
4
5
 
     strcpy(ext,argv);
     ou
     for(i=0;i<strlen(argv[2]);i++)
        *(ext+i) = argv[2][1];

la premiere etant pratique fait ca pour toutes tes chaines et apres dis nous si tu as toujours l'erreur?

**Emmanuel Delahaye** · 23/12/2008, 11h38

Envoyé par titou624

Tout est dit dans le titre à la compilation aucun problème mais lors de l'exécution de mon programme.
Je vous met le code et les explications. Ce code à pour but de chercher les fichiers comportant l'extension passée en paramètre puis de modifier le contenu en le cryptant en rot13. Le code n'est pas encore finni, pour l'instant je veux juste qu'il m'affiche les fichiers trouvés. Il m'affiche bien les fichiers mais ensuite il m'affiche "erreur de segmentation".

Ce code (j'ai fait quelques remarques)

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
 
/*
crypte rep_depart extension nbMaxFils (Nb maximum de fils qui cypteront chacun des fichiers trouvés)
 
tube :
	-> le père ouvre les chemin du fichier a traiter
	-> les fils signalent qu'ils ont terminé
 
Tous les fichiers possédant cette extension effectue un cryptage rot13
	-> par un processus fils
	-> communication par tube
*/
 
#include <sys/stat.h>
#include <sys/types.h>
#include <stdlib.h>
#include <stdio.h>
#include <dirent.h>
#include <string.h>
 
#define MAXNAME 100
 
/*
	on créé le tube en premier
*/
int p[2];
 
int main (int argc, char **argv)
{
   int ret;
/* Test des arguments */
   if (argc != 4)
   {
      printf ("Usage: crypte rep_depart ext nbProc\n");
      ret = EXIT_FAILURE;
   }
   else
/* Déclaration des variables */
   {
/* parametres dela ligne de commande */
      char *nomRep = argv[1];
      char *ext = argv[2];
/* On converti le nombre de processus
     int nbProc = atoi (argv[3]);
*/
 
/* ouverture du dossier */
      DIR *rep = opendir (nomRep);
 
/* Test si le dossier s'ouvre correctement */
      if (!rep)
      {
/* On signale l'erreur */
         perror ("opendir");
         ret = EXIT_FAILURE;
      }
      else
      {
         struct dirent *ent;
/* Tant qu'il ya des fichiers */
         while ((ent = readdir (rep)) != NULL)
         {
 
            char nom[MAXNAME];
/* On construit l'adresse du fichier */
/* -ed- risque de debordement ... */
            strcpy (nom, nomRep);
 
/* Si le fichier est différent de ./ ou ../ */
            if (strcmp (ent->d_name, ".") != 0
                && strcmp (ent->d_name, "..") != 0)
            {
               strcat (nom, "/");
               strcat (nom, ent->d_name);
 
/* On récupère les informations sur le fichier */
/* lstat permet d'éviter de compter les liens symbolique
 
-ed- (inconnu sur ma machine)
 
*/
               {
                  struct stat buf;
                  if (stat (nom, &buf) != -1)
                  {
/* Si c'est un fichier */
                     if (S_ISREG (buf.st_mode))
                     {
                        char *tmp = strrchr (ent->d_name, '.');
                        tmp++;
                        if (strcmp (tmp, ext) == 0)
                        {
                           printf ("%s \n", nom);
                        }
                     }
                  }
               }
            }
         }
/* On ferme le repertoire */
         closedir (rep);
         ret = EXIT_SUCCESS;
      }
   }
   return ret;
}

avec les paramètres

Code :

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

. txt 1

fonctionne sans problèmes sous Windows :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
./data.txt
./dico.txt
./fable.txt
./fichier.txt
./groupes.txt
./in.txt
./niveaux.txt
 
Process returned 0 (0x0)   execution time : 0.018 s
Press any key to continue.

J'ai trouvé le bug. Si il y a des fichiers sans extension, c'est le drame avec :

Code :

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

1
2
3
4
5
6
7
 
                        char *tmp = strrchr (ent->d_name, '.');
                        tmp++;
                        if (strcmp (tmp, ext) == 0)
                        {
                           printf ("%s \n", nom);
                        }

il faut

Code :

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

1
2
3
4
5
6
7
8
9
 
                        char *tmp = strrchr (ent->d_name, '.');
                        if (tmp != NULL)
                        {
                           if (strcmp (tmp + 1, ext) == 0)
                           {
                              printf ("%s \n", nom);
                           }
                        }

**titou624** · 23/12/2008, 11h49

Merci pour ces information précieuses j'ai testé ces deux méthodes mais toujours cette même erreur ....
Par soucis de simplicité j'ai allégé le code. Ce code cherche uniquement les fichiers avec l'extension donnée en paramètre mais je pense qu'il y a pas mal d'erreurs car il ne fonctionne pas et me marque toujours cette erreur de segmentation qui je l'avoue commence a e mettre les nefs dans un sale état....
Lorsque l'on fait un strcpy faut-il faireun malloc avant ou pas pour la variable de destination ?

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
#include <sys/stat.h>
#include <sys/types.h>
#include <stdlib.h>
#include <stdio.h>
#include <dirent.h>
#include <string.h>
 
#define MAXNAME 100
 
int main(int argc, char **argv){
 
	if(argc!=3){
		printf("Usage: find rep_depart ext \n");
		exit(2);
	}
 
// initialisation des variables
	struct stat buf;
	struct dirent *ent;
	char nom[MAXNAME];
	char *ext = NULL;
	char *nomRep = NULL;
	DIR *rep = NULL;
 
// On stock l'extension en variable
	strcpy(ext,argv[2]);
 
// On stock le nom du repertoire
  strcpy(nomRep,argv[1]);
 
//ouverture du dossier
	rep = opendir(nomRep);
 
//Test si le dossier s'ouvre correctement
	if(!rep){	
// On signale l'erreur
		perror("opendir");
	}
 
// Tant qu'il ya des fichiers
	while(ent=readdir(rep))
	{		
 
// On construit l'adresse du fichier
			strcpy(nom,nomRep);
 
// Si le fichier est différent de ./ ou ../
			if(strcmp(ent->d_name,".") && strcmp(ent->d_name,"..")){
				strcat(nom,"/");
				strcat(nom,ent->d_name);
 
// On récupère les informations sur le fichier
 
// lstat permet d'éviter de compter les liens symbolique
				if(lstat(nom,&buf)!=-1){
 
// Si c'est un fichier
						if(S_ISREG(buf.st_mode)){							
								char *tmp = strrchr(ent->d_name,(int)'.');
								tmp++;
								printf("%s = %s ???\n",tmp,ext); 
								if(!strcmp(tmp,ext)){
									printf("OUI %s \n",nom);
 
								}
						}	
				}
			}
	}
// On ferme le repertoire
	closedir(rep);
}

**Emmanuel Delahaye** · 23/12/2008, 11h53

Envoyé par titou624

Merci pour ces information précieuses j'ai testé ces deux méthodes mais toujours cette même erreur ....

J'ai indiqué où était le bug...

**titou624** · 23/12/2008, 12h07

Merci pour ta réponse Emmanuel !
J'ai testé ton code je pense que c'est le problème car je suis sur linux ^^ désormais je crois que c'est une erreur de syntaxe.
Voici le nouveau 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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
#include <sys/stat.h>
#include <sys/types.h>
#include <stdlib.h>
#include <stdio.h>
#include <dirent.h>
#include <string.h>
 
#define MAXNAME 100
 
int main(int argc, char **argv){
 
	if(argc!=3){
		printf("Usage: find rep_depart ext \n");
		exit(2);
	}
 
// initialisation des variables
	struct stat buf;
	struct dirent *ent;
	char nom[MAXNAME];
	char *ext = NULL;
	char *nomRep = NULL;
	DIR *rep = NULL;
 
// On stock l'extension en variable
	strcpy(ext,argv[2]);
 
// On stock le nom du repertoire
  strcpy(nomRep,argv[1]);
 
//ouverture du dossier
	rep = opendir(nomRep);
 
//Test si le dossier s'ouvre correctement
	if(!rep){	
// On signale l'erreur
		perror("opendir");
	}
 
// Tant qu'il ya des fichiers
	while(ent=readdir(rep))
	{		
 
// On construit l'adresse du fichier
	    strcpy(nom,nomRep);
 
// Si le fichier est différent de ./ ou ../
	    if(strcmp(ent->d_name,".") && strcmp(ent->d_name,"..")){
		        strcat(nom,"/");
			strcat(nom,ent->d_name);
 
// On récupère les informations sur le fichier
 
// lstat permet d'éviter de compter les liens symbolique
	  if(lstat(nom,&buf)!=-1){
 
// Si c'est un fichier
	     if(S_ISREG(buf.st_mode)){							
  	        char *tmp = strrchr (ent->d_name, '.');
                if (tmp != NULL){
                   if (strcmp (tmp + 1, ext) == 0){
                      printf ("%s \n", nom);
                   }
                }
	     }	
         }
     }
  }//fin du while
 
// On ferme le repertoire
	closedir(rep);
}

j'essaye de débugger comme je puex ....
En tout cas merci à vous pour l'interet que vous me portez ^^

**Emmanuel Delahaye** · 23/12/2008, 13h10

Envoyé par titou624

Merci pour ta réponse Emmanuel !
J'ai testé ton code je pense que c'est le problème car je suis sur linux ^^

dans mon code, j'ai remplacé lstat() par stat() ...

Ton code (avec stat) provoque un crash chez moi (Windows). J'avais pourtant publié un code sécurisé... Je ne vais pas tout refaire, tu as tous les éléments.

Purée, c'est quoi ça ?

Code :

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

1
2
3
4
5
6
 
      char *ext = NULL;
<...>
 
/* On stock l'extension en variable */
      strcpy (ext, argv[2]);

Tu stockes où ? Et pourquoi tu fais une copie ? Il suffit de stocker l'adresse comme tu le faisais avant. Idem avec l'autre paramètre.

Ces 2 corrections faites, le programme fonctionne correctement.

**titou624** · 23/12/2008, 13h15

Oui merci tu as déjà fais beaucoup je vais me débrouiller tout seul ^^.
Merci à tous !

**titou624** · 23/12/2008, 14h43

Bravo à tous mon code marche parfaitement. J'en ai eu tellement marre que j'ai tout recommencé en suivant les précieux conseils de tout le monde. Et ça a marché du premier coup !
J'ai vraiment apprécié l'aide de tous les membres et dès que j'aurai un peu plus d'expérience je compte bien en faire partie !
Bonne journée à tous !

Erreur de segmentation : à cause d'un strcmp()

C

Discussions similaires

Partager

Partager