seg fault avant la 1ere ligne du main !!??

seg fault avant la 1ere ligne du main !!?? - C - Programmation

Marsh Posté le 25-11-2004 à 00:43:07    

voilà, je suis entrain de recoder la commande Unix "pwd" en C, avec des opendir, des readdir et une petite pile FILO maison, et, comme je débute en C, je ne comprend pas trop pourquoi j'ai un seg fault au lancement du prog, alors que je n'ai aucune erreur à la compile.
 
j'ai mis un printf à la premiere ligne de mon main et il ne me l'affiche pas.
 
est ce qu'il parcourt les autres fonctions avant mon main et qu'il seg fault à cause d'une d'entre elles?
 
comment savoir où se situe le problème?

Reply

Marsh Posté le 25-11-2004 à 00:43:07   

Reply

Marsh Posté le 25-11-2004 à 00:49:43    

gcc -ggdb machin.c
gdb ./a.out
> run
> bt

Reply

Marsh Posté le 25-11-2004 à 00:59:24    

connaissais pas!
 
merci pour ce conseil plus qu'utile!

Reply

Marsh Posté le 25-11-2004 à 01:22:57    

euh, je suppose que quand le debugger m'indique une erreur dans strlen, je suppose obligatoirement que c'est un problème par rapport à ce que je lui passe en paramètre?
 
pourtant :
 
 
strlen(stack->doss);
 
avec
typedef struct s_stack
{
 char * doss;
 struct s_stack *next;
} t_stack
 
et
t_stack *stack;

Reply

Marsh Posté le 25-11-2004 à 01:35:28    

Tu n'aurais pas bouffer le caractère nul dans la chaine que tu lui passe en paramètre?
strlen va parcourir toute la zone mémoire à la recherche d'un 0, en déborder -> SIGSEGV
Utilise strnlen


Message édité par manatane le 25-11-2004 à 01:37:44
Reply

Marsh Posté le 25-11-2004 à 01:52:08    

merci pour ton aide
 
en fait j'ai fait un malloc pour mon char * doss en prenant bien strlen de ma chaine de caractère + 1 (pour le \0)
 
je pense pas que ça vienne de là finalement...
maintenant il me parle d'une erreur sur strcpy (!) donc je regarde ça...
 
 
 
pffff, quelle idée de partir en école d'info :sarcastic:

Reply

Marsh Posté le 25-11-2004 à 02:00:20    

bon, j'ai testé autre chose et il me trouve tjs une erreur :
 

Code :
  1. int cpt=0;
  2. cpt=strlen(stack->doss);


 
et j'ai toujours une erreur sur ma ligne du strlen :'(
 
ça peut pas etre un problème avec la forme de stack->doss ?
 
je comprend plus rien là :'(

Reply

Marsh Posté le 25-11-2004 à 02:04:12    

résolu!
 
j'ai rajouter un
 

Code :
  1. if (stack->doss)


 
l'erreur était gérée differement avant mais comme ça marche, enfin, maintenant il me trouve d'autres erreurs :sarcastic:


Message édité par ACYT le 25-11-2004 à 02:04:45
Reply

Marsh Posté le 25-11-2004 à 09:04:29    

ACYT > si ton code n'est pas trop gros tu peux toujours le poster.


---------------
Le site de l'année :D (XHTML 1.0 strict) : http://darkoli.free.fr/index.html
Reply

Marsh Posté le 25-11-2004 à 09:39:50    

euh tu l'alloues comment stack ?

Reply

Marsh Posté le 25-11-2004 à 09:39:50   

Reply

Marsh Posté le 25-11-2004 à 13:13:23    

je pense effectivement que le plus simple pour que vous puissiez comprendre serait de poster le code mais je suis sûr que vous allez trouver plein plein de chose pas bien dedans :whistle:
 
il fait environ 110 lignes, c'est trop?

Reply

Marsh Posté le 25-11-2004 à 13:53:54    

bon ben, voilà le code : (je suis ouvert à toute critique qui me permettrait de mieux coder!!)
 
désolé je n'ai pas encore commenté le code par contre  
 

Code :
  1. #include <dirent.h>
  2. #include <stdio.h>
  3. #include <stdlib.h>
  4. #include <unistd.h>
  5. #include <string.h>
  8. typedef struct s_stack
  9. {
  10.   char *doss;
  11.   struct s_stack *next;
  12. } t_stack;
  14. t_stack * create_stack ()
  15. {
  16. t_stack *stack;
  17. stack = 0;
  18. return stack;
  19. }
  22. t_stack * push(t_stack *stack, char * data)
  23. {
  25. t_stack * elt;
  26. elt = malloc(sizeof(t_stack));
  27. elt->doss = malloc((strlen(data)+1)*sizeof(char));
  28. strcpy(elt->doss, data);
  29. elt->next=stack;
  30. stack=elt;
  31. // printf("j'ai bien pushé\n" );
  32. return (stack);
  33. }
  36. t_stack * pop(t_stack *stack)
  37. {if (stack != 0)
  38. {
  39. // printf("je pop" );
  40. char * tmp;
  41. int cpt=0;
  42. if (stack->doss)
  43. cpt=strlen(stack->doss);
  45. tmp = malloc((cpt+1)*sizeof(char));
  46. t_stack *tmp_free;
  47. if(stack->doss)
  48. tmp = strcpy(tmp , stack->doss);
  49. tmp_free = stack;
  50. stack=(stack->next);
  51. free(tmp_free);
  52. // printf("j'ai bien popé" );
  53. printf("/%s",tmp);
  54. free(tmp);
  55. return(stack);
  56. }
  57. return stack;
  58. }
  61. void aff(t_stack *stack)
  62. {
  63. while(stack != 0)
  64. {
  65. //  printf(" %d ",stack);
  66.   stack = pop(stack);
  67. //  printf(" %d ",stack);
  68.   if (stack==0);
  69. printf(" stack est deja à 0 " );
  70. }
  71. }
  72.  
  74. t_stack * pwd(int ino, t_stack *stack)
  75. {
  76.   struct dirent *s_dir;
  77.   DIR *dir;
  79.   chdir(".." );
  80.   s_dir = readdir(opendir("." ));
  81.   if (s_dir->d_ino != (readdir(opendir(".." )))->d_ino)
  82. pwd(s_dir->d_ino, stack);
  83.   dir = opendir("." );
  84.   while ((s_dir = readdir(dir)) && (s_dir->d_ino !=(unsigned)ino || *(s_dir->d_name) == '.'))
  85. ;
  86.   if (s_dir)
  87.   {
  88. stack= push(stack, s_dir->d_name);
  90. if(chdir(s_dir->d_name) < 0)
  91.  exit(1);
  92.   }
  93.   else
  94. printf("/" );
  96. return (stack);
  97. }
  99. int main ()
  100. {
  101.   t_stack *stack;
  103.   stack = create_stack();
  104.   stack = pwd(readdir(opendir("." ))->d_ino, stack);
  105. //  printf("donc je passe par là " );
  106.   aff(stack);
  107.   printf("\n" );
  108.   return (0);
  109. }


 
il y a quelques printf par ci par là qui m'ont permis de localiser l'erreur...

Reply

Marsh Posté le 25-11-2004 à 14:12:27    

ok problème résolu
j'avais juste oublié de récupérer stack à la ligne 83  :whistle:
 
 
 
me reste plus qu'à afficher le résultat dans le bon ordre

Reply

Marsh Posté le 25-11-2004 à 14:13:35    

c'est une blague ton create_stack ?

Reply

Marsh Posté le 25-11-2004 à 14:24:03    

euh.... oui?
après reflexion c'est sûr qu'elle sert à rien mais en fait au début j'étais parti sur une liste chaînée et j'initialisais beaucoup plus de chose dans cette fonction...
 
par contre je dois vraiment avoir fait une pile de merde pour qu'elle arrive à me ressortir les valeurs dans le mauvais sens :pfff:

Reply

Marsh Posté le 25-11-2004 à 14:27:32    

et puis, c'est NULL et pas 0

Reply

Marsh Posté le 25-11-2004 à 14:35:43    

un doute me prend : pourquoi si je rajoute un printf("/%s", s_dir->d_name);  
 
avant la ligne 89, il m'affiche directement le resultat sans passer par ma pile??
 
j'ai fait tout ça pour rien??
 
:'(


Message édité par ACYT le 25-11-2004 à 14:36:40
Reply

Marsh Posté le 25-11-2004 à 14:37:14    

ACYT a écrit :

j'ai fait tout ça pour rien??


Tu veux dire que tu aurais pu utiliser getcwd ?  
   http://man.he.net/man3/getcwd
 
edit: en plus, je crois que ton truc ne marche pas si le nom d'un de tes répertoires commence par un '.'


Message édité par Lam's le 25-11-2004 à 14:38:09
Reply

Marsh Posté le 25-11-2004 à 14:48:53    

bon ben voilà, je viens de supprimer ma pile et tout fonctionne avec juste un petit printf bien placé
 
j'ai la rage, j'ai bossé jusqu'à 3h30 pour ça ce matin
 
voilà le code final :
 
edit : en fait je le remettrai quand le projet sera rendu, pas envie que des petits malins le pompe entre temps
 
 
en tout cas merci à tous pour votre aide!
 
:hello:


Message édité par ACYT le 25-11-2004 à 16:38:16
Reply

Marsh Posté le 02-12-2004 à 23:30:27    

ACYT a écrit :

bon ben, voilà le code :  


Voici une version simplifiée et corrigée:  

Code :
  1. /* -ed-
  2. * Commentaires C90
  3. * suppression accents
  4. * Re-indentation
  5. * Suppression du code non portable
  6. * Test du mecanisme de stack.
  7. * Mise au point des liberation manquantes.
  8. */
  10. #include <stdio.h>
  11. #include <stdlib.h>
  12. #include <string.h>
  14. typedef struct s_stack
  15. {
  16.    char *doss;
  17.    struct s_stack *next;
  18. }
  19. t_stack;
  21. t_stack *push (t_stack * stack, char *data)
  22. {
  23.    t_stack *elt = malloc (sizeof (t_stack));
  25.    if (elt != NULL)
  26.    {
  27.       elt->doss = malloc (strlen (data) + 1);
  28.       strcpy (elt->doss, data);
  29.       elt->next = stack;
  30.       stack = elt;
  31.    }
  32.    return stack;
  33. }
  35. t_stack *pop (t_stack * stack)
  36. {
  37.    if (stack != NULL)
  38.    {
  39.       if (stack->doss)
  40.       {
  41.          printf ("/%s\n", stack->doss);
  42.       }
  44.       {
  45.          t_stack *tmp_free = stack;
  47.          stack = stack->next;
  49.          free (tmp_free->doss);
  50.          free (tmp_free);
  51.       }
  52.    }
  53.    return stack;
  54. }
  56. void aff (t_stack * stack)
  57. {
  58.    while (stack != NULL)
  59.    {
  60.       stack = pop (stack);
  62.       if (stack == NULL)
  63.       {
  64.          printf (" stack est deja a 0\n" );
  65.       }
  66.    }
  67. }
  69. int main (void)
  70. {
  71.    t_stack *stack = NULL;
  73.    stack = push (stack, "1" );
  74.    stack = push (stack, "2" );
  75.    stack = push (stack, "3" );
  77.    aff (stack);
  79.    return 0;
  80. }


Message édité par Emmanuel Delahaye le 03-12-2004 à 08:22:55
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed