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.

**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);
                           }
                        }

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

C

Vue hybride

Discussions similaires

Partager

Partager