Discussion :

[Atharendil]

Bonjour,

en entrant un certain nombre de données dans un tableau, je me retrouve joyeusement avec un sympathique segmentation fault dont je n'arrive pas à trouver la cause.

Voici comment le pointeur dans lequel je rentre les données est instancié (sachant que je ne peux pas modifier cette partie du 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
 
TYPEMRI3D ***cr_mri_3d(UINT width, UINT height, UINT depth)
{
  TYPEMRI3D *t1, ***t3;
  unsigned int       i, j;
 
  if((t3 = CALLOC((width + 1), TYPEMRI3D**)) == NULL)
    return (TYPEMRI3D***)NULL;
 
  if((t3[0] = CALLOC((width * height), TYPEMRI3D*)) == NULL)
    {
      FREE(t3);
      return (TYPEMRI3D***)NULL;
    }
 
  for(i = 1 ; i < width ; i++)
    t3[i] = t3[i - 1] + height;
 
  if((t3[0][0] = CALLOC((width * height * depth), TYPEMRI3D)) == NULL)
    {
      FREE(t3[0]);
      FREE(t3);
      return (TYPEMRI3D***)NULL;
    }
 
  t1 = t3[0][0];
 
  for(i=0 ; i < width ; i++)
    {
      for(j = 0 ; j < height ; j++)
        {
      t3[i][j] =  t1;
      t1       += depth;
        }
    }
 
  return t3;
}

TYPEMRI3D est en fait un type short. Cette fonction renvoie donc un tableau à 3 dimensions qui fera partie d'une structure dont le champ en question est nommé mri.

Voilà maintenant mon propre code, dans lequel j'entre des données dans le tableau mri :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
for(i=0; i<image->width; i++){
        for(j=0; j<image->height; j++){
            for(k=0; k<image->depth; k++){
                image->mri[i][j][k] = (*(TYPEMRI3D*)image_data);
 
                if(i+1 != image->width){
                    image_data += image->bitppixel / 8; //on incrémente du nombre de bits par voxel
                }
            }
        }
    }

image->bitppixel / 8 donne sizeof(short) et width, height et depth ici sont inférieurs ou égaux aux dimensions respectives du tableau.

image_data, qui référence les données (de type short) à stocker dans mri, est en fait un pointeur de type void * instancié comme tel :

Code :

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

(void *)calloc(1,nim->nbyper * nim->nvox);

nbyper étant égal à sizeof(short) et nvox correspondant à image->width * image->height * image->depth.

D'après mon debugger, l'erreur de segmentation a lieu lors de l'affectation des données dans mri.

Je remercie mes futurs sauveteurs

[Emmanuel Delahaye]

Voici comment le pointeur dans lequel je rentre les données est instancié (sachant que je ne peux pas modifier cette partie du code) :

Donne du code compilable complet et réduit au minimum qui montre le défaut.

[PRomu@ld]

D'après mon debugger, l'erreur de segmentation a lieu lors de l'affectation des données dans mri.

C'est donc ici : (???)

Code :

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

mage->mri[i][j][k] = (*(TYPEMRI3D*)image_data);

mri je suppose que tu l'obtiens comme ceci :

Code :

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

mri = cr_mri_3d(w,h,d);

Testes tu la valeur de mri avant d'appeler tes trois boucles ?

Je remplacerai ceci :

Code :

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

i+1 != image->width

par ceci :

Code :

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

i+1 < image->width

Ton segfault aparait il lors de la première itération de la boucle ou bien après ?

[Atharendil]

Merci à vous pour vos réponses et désolé de répondre si tard (WE oblige ) !

Emmanuel, je voudrais bien mettre du code compilable mais ça n'va pas être réellement possible vu que j'utilise, rien que pour ces petits bouts de code, plusieurs librairies d'entrées/sorties... et il faudrait aussi que je mette le code intégral de la sauvegarde (puisque le code plus haut correspond en fait au chargement d'une image).

PRomuald, pour te répondre, il est vrai que je ne teste pas la valeur de mri avant de rentrer les données dedans, peut-être devrais-je le faire, je vais essayer cela.

C'est donc ici : (???)

Code :

mage->mri[i][j][k] = (*(TYPEMRI3D*)image_data);

Oui, c'est bien ici que l'erreur de segmentation se produit et bien après la 1ère itération de la boucle (j'ai même l'impression que c'est vers la fin, voire après la fin, d'où l'erreur).