structure imbriquée malloc problème

Version imprimable

Bonsoir à tous,
j'éprouve quelques souci quant à l'allocation dynamique de mémoire sur des structures imbriquées.

voici mon code

Code:

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
typedef struct
{
  char *intitule;
  char *value;
} Parametre;
 
typedef struct
{
  char *intitule;
  Parametre *parametres;
} Commande;
 
Commande * parse_commande ( char * buffer )
{
  ...
  Commande *com;
  ...
  com = (Commande *) malloc ( sizeof ( Commande ) ); // Allocation de la mémoire pour la commande
  com->parametres = (Parametre *) malloc ( 0 ); // Allocation de la mémoire du tableau de param
  ...
  com->intitule = (char *) malloc ( sizeof (char) * size_intitule ); 
  strncpy ( com->intitule, pt_com1, size_intitule );
  ...
 
  for ( ... )
    {
      ...
      com->parametres = (Parametre *) realloc ( com->parametres, i * sizeof ( Parametre ) );
      ...       
      intitule = (char *) malloc ( sizeof ( char ) * size_intitule_param );
      com->parametres[i].intitule = intitule;
      strncpy ( com->parametres[i].intitule, pt_com1, size_intitule_param );
      ...
      value = (char *) malloc ( sizeof ( char ) * size_value_param );
      ...
      strncpy ( com->parametres[i].value, pt_com1 + size_intitule_param, size_value_param );
 
    }
 
  return com;
}

Ce code plante à l'exécution sur le realloc ... Je pense rater qqch mais la je ne vois pas !
Si quelqun a une idée

nb : si je n'alloue pas la mémoire des variables de la structure interne à Commande, le code se déroule sans problème... si ça peut aider

26/02/2007, 02h21
tuxout
Je viens d'essayer cela qui marche.
Code:

1 2 3 4 5 6 typedef struct { char *intitule; Parametre **parametres; int n; } Commande;
Cela donne un tableau de pointeur.
Ce qui me "turlupine", c'est pourquoi est ce que *parametres ne marche pas, puisque toutes mes autres expériences sur des chaines caractères fonctionnent... Ou alors ça marche pas, et mon programme explosera un jour sur un problème d'allocation mémoire ...
26/02/2007, 02h37
souviron34

ben ton premier alloc tu lui donnes une taille de zéro.. Donc il n'alloue rien, donc le pointeur n'est pas défini.
Quand ensuite tu fais un realloc, il veut un pointeur déjà défini en entrée...

Il suffit de faire ton premer alloc à 1..
26/02/2007, 06h49
granquet

enleve ton premier malloc

Citation:

com->parametres = (Parametre *) malloc ( 0 ); // Allocation de la mémoire du tableau de param

une taille de 0 ... je sais pas ce que ça donne :aie:
remplace le par un

Code:

com->parametres = NULL;

comme ça tu passe NULL a ton realloc, (qui deviens equivalent a un malloc au premier tour de boucle)
ensuite l'utilisation de realloc est dangereuse.
si ton realloc echoue, tu perd l'addresse de la zone memoire precedente, il faut passer par un pointeur temporaire, verifier qu'il n'est pas null, et le refiler a com->parametres.
details: http://emmanuel-delahaye.developpez....es.htm#realloc

---

ah oui:
-les cast devant les malloc, ça sert a rien.
-sizeof(char) c'est 1, par definition.

et dans un souci d'efficacité, jte conseillerais de compter d'abord le nombre d'elements qu'il te faut.
allouer l'espace, et enfin refaire une boucle pour traiter la chose.