Discussion :

[Pitou5464]

J'ai un code qui permet de lister tout les repertoires et les fichiers de mon repertoire passe en parametre. Le probleme c'est que je voudrais qu'il fasse la meme chose lorsqu'il tombe sur repertoire et non sur un fichier. Donc je voudrais rappeler la fonction mais il seg fault.

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
int     do_ls(char *rep)
{
  DIR           *dir;
  struct dirent *direntry;
  struct stat   s;
 
  dir = opendir(rep);
  while ((direntry =readdir(dir)) != NULL)
    {
      stat(direntry->d_name, &s);
      if ((s.st_mode & S_IFMT) == S_IFDIR)
        {
          do_ls(direntry->d_name);
        }
      else
        {
          aff_type(s.st_mode);
          aff_perm(s.st_mode);
          my_putchar(' ');
          my_putnbr(s.st_nlink);
          my_putchar(' ');
          my_putstr(getpwuid(s.st_uid)->pw_name);
          my_putchar(' ');
          my_putstr(getgrgid(s.st_gid)->gr_name);
          my_putchar(' ');
          my_putnbr(s.st_size);
          my_putchar(' ');
          print_date(s.st_mtime);
          my_putchar(' ');
          my_putstr(direntry->d_name);
          my_putchar('\n');
        }
    }
 
}

[Patriarch24]

Sors ton debugger et c'est parti.

dir = opendir(rep);

Vérifie le retour. (dir != NULL).

[Pitou5464]

merci pour ta reponse mais je crois pas que ce soit ici que ca peche. par contre si j'enleve cette ligne ca marche mais plus de recursivite...

Code :

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

do_ls(direntry->d_name);

[Médinoc]

Le premier répertoire trouvé ne serait-il pas "." ?

[Pitou5464]

oui enfin je le passe en argument, mais de toute facon dans nimporte quelle repertoire il y aura toujours le point non?

[zooro]

oui enfin je le passe en argument, mais de toute facon dans nimporte quelle repertoire il y aura toujours le point non?

Oui, mais si le "." est passé indéfiniment, tu finiras sans doute par manquer de mémoire...
Il faut tester le répertoire avant l'appel récursif, et ne pas le faire dans le cas de "." et de "..".