destruction de ma structure

**salseropom** · 20/11/2006, 11h24

Bonjour, je n'arrive pas à expliquer la cause de mon segmentation fault.

Voici mon fichier .c

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
 
#include<stdio.h>
#include<stdlib.h>
 
#include "distribution.h"
 
#define MEMERROR fprintf (stderr,"%s %d : Allocation Memory Error\n", __FILE__,__LINE__)
 
Distribution * CreateEmptyDistribution(void)
{
  Distribution * d=malloc(sizeof(*d));
  if(d==NULL)
  {
    MEMERROR;
    exit(EXIT_FAILURE);
  }
 
  d->name=NULL;
 
  d->param=malloc(sizeof(*d->param));
  if(d->param==NULL)
  {
    MEMERROR;
    exit(EXIT_FAILURE);
  }
 
  return d;
}
 
void DestroyDistribution(Distribution ** d)
{
  free((*d)->name);  (*d)->name=NULL;
  free((*d)->param); (*d)->param=NULL;
  free(*d); *d=NULL;
}

Tout à la fin de mon main, je fais

Code :

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

1
2
 
DestroyDistribution(&d);

et c'est à ce niveau qu'il y a mon segmentation fault. Pourquoi ? Dans mon code, je fais

Code :

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

1
2
3
4
5
 
d->name=strdup(name);
d->param->eps=0.1;
d->param->delta=1e-9;
d->param->var=1.;

Rien de bien sorcier en somme.
Mais je doute que mon erreur vienne de là. Depuis ce matin, je mets des pointeurs de fonctions un peu partout, mais je n'ai jamais touché à la variable d (ma distribution). Est-ce qu'une erreur de pointeur peut se "déplacer" pour "atterir" à mon DestroyDistribution(&d); ?
Pourtant, je suis quasiment sûr que ma fonction DestroyDistribution est correctement écrite...

Merci de votre aide.

**Emmanuel Delahaye** · 20/11/2006, 11h48

Envoyé par salseropom

Bonjour, je n'arrive pas à expliquer la cause de mon segmentation fault.

Grâce à ma boule de cristal, j'ai pu reconstituer ton code. Il est correct et il fonctionne. L'erreur est ailleurs. (Typiquement, débordement de tableau alloué)

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
#include<stdio.h>
#include<string.h>
#include<stdlib.h>
#include <assert.h>
 
struct param
{
   double eps;
   double delta;
   double var;
};
 
typedef struct
{
   char *name;
   struct param *param;
}
Distribution;
 
#define MEMERROR fprintf (stderr,"%s %d : Allocation Memory Error\n", __FILE__,__LINE__)
 
Distribution *CreateEmptyDistribution (void)
{
   Distribution *d = malloc (sizeof *d);
   if (d == NULL)
   {
      MEMERROR;
      exit (EXIT_FAILURE);
   }
 
   d->name = NULL;
 
   d->param = malloc (sizeof *d->param);
   if (d->param == NULL)
   {
      MEMERROR;
      exit (EXIT_FAILURE);
   }
 
   return d;
}
 
void DestroyDistribution (Distribution ** d)
{
   free ((*d)->name), (*d)->name = NULL;
   free ((*d)->param), (*d)->param = NULL;
   free (*d), *d = NULL;
}
 
int main (void)
{
   Distribution *d = CreateEmptyDistribution ();
 
   if (d != NULL)
   {
      d->name = strdup ("nom");
      d->param->eps = 0.1;
      d->param->delta = 1e-9;
      d->param->var = 1.;
 
      DestroyDistribution (&d);
   }
 
   assert (d == NULL);
   return 0;
}

Par contre, je recommande ceci :

Code :

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

1
2
3
4
5
6
7
8
9
void DestroyDistribution (Distribution ** d)
{
   if (d != NULL && *d != NULL)
   {
      free ((*d)->name), (*d)->name = NULL;
      free ((*d)->param), (*d)->param = NULL;
      free (*d), *d = NULL;
   }
}

Ca peut te sauver la vie si tu passes un pointeur NULL ou l'adresse d'un pointeur valant NULL.