[VC++2005] malloc et realloc

Bonjour ,

dans la portion de code suivante:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

struct KprotoPckt{ uint8_t Type; // Type de trame uint16_t Checksum; // Reçue du Réseau uint16_t DataLength; // Reçue du Réseau uint8_t Serial[SERIAL_NUMBER_LENGTH]; // Numéro de série du produit uint8_t *Data; // Les données transportées };   if(KprotoPckt->Data) KprotoPckt->Data=realloc(KprotoPckt->Data,KprotoPckt->DataLength * sizeof(uint8_t *)); else KprotoPckt->Data=malloc(KprotoPckt->DataLength * sizeof(uint8_t *));   if(KprotoPckt->Data == NULL){ datation(); printf("Error: can't allocate memory for a RX packet !!! \n"); free(KprotoPckt->Data); return; }

---

j'ai un plantage au niveau du realloc => Errreur allocation de memoire

Je libere bien la memoire apres traitemement comme ce-ci:

Code:

---

1 2 3 4 5

if(KprotoPckt.Data != NULL){ free(KprotoPckt.Data); KprotoPckt.Data=NULL; }

---

d'ou peux provenir le probleme ?
est ce une incompatibilite de mon systeme ? je suis sur XP ?
J'utilise Visual Studio 2005, peut etre qu'il existe une directive de compilation pour eviter cela ?
que veux dire le message exactement ? est ce que ca veut dire qu'il n'arrive plus a allouer du tas au niveau memoire ?
est ce au niveau de la memoire vive?
Est ce un probleme de cadrage au niveau memoire ?
j'ai 2go de ram.

Merci de votre aide.

Bonjour,

1. Tu n'es pas obligé de faire ton test pour malloc/realloc : La norme dit que realloc(NULL, ...) doit se comporter comme malloc(...).
2. Pareil pour le free: La norme dit que free(NULL) ne doit pas planter.
3. Que veux-tu dire exactement par "plantage au niveau du realloc => Errreur allocation de memoire" ?
4. Vérifie que DataLength possède une valeur correcte.

Citation:

---

Envoyé par andromeda

dans la portion de code suivante:

---

Ton code est incomplet. J'ai fait quelques hypothèses. Ce code fonctionne :

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 42 43 44 45 46 47 48 49 50 51 52

#include <stdio.h> #include <stdlib.h> #include <stdint.h>   #define SERIAL_NUMBER_LENGTH 32   struct KprotoPckt { uint8_t Type; // Type de trame uint16_t Checksum; // Reçue du Réseau uint16_t DataLength; // Reçue du Réseau uint8_t Serial[SERIAL_NUMBER_LENGTH]; // Numéro de série du produit uint8_t *Data; // Les données transportées };   void datation (void) { puts ("datation"); }   static void f (struct KprotoPckt *KprotoPckt) { if (KprotoPckt->Data) KprotoPckt->Data = realloc (KprotoPckt->Data, KprotoPckt->DataLength * sizeof (uint8_t *)); else KprotoPckt->Data = malloc (KprotoPckt->DataLength * sizeof (uint8_t *));   if (KprotoPckt->Data == NULL) { datation (); printf ("Error: can't allocate memory for a RX packet !!! \n"); free (KprotoPckt->Data); return; } }   int main (void) { struct KprotoPckt KprotoPckt = { 0, 0, 0, {0}, 0 }; f (&KprotoPckt);   if (KprotoPckt.Data != NULL) { free (KprotoPckt.Data); KprotoPckt.Data = NULL; }   return 0; }

---

Mais comme l'a très justement signalé Medinoc, il est inutilement complexe...
Ceci est suffisant :

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 42 43 44

#include <stdio.h> #include <stdlib.h> #include <stdint.h>   #define SERIAL_NUMBER_LENGTH 32   struct KprotoPckt { uint8_t Type; // Type de trame uint16_t Checksum; // Reçue du Réseau uint16_t DataLength; // Reçue du Réseau uint8_t Serial[SERIAL_NUMBER_LENGTH]; // Numéro de série du produit uint8_t *Data; // Les données transportées };   void datation (void) { puts ("datation"); }   static void f (struct KprotoPckt *KprotoPckt) { KprotoPckt->Data = realloc (KprotoPckt->Data, KprotoPckt->DataLength * sizeof *KprotoPckt->Data);   if (KprotoPckt->Data == NULL) { datation (); printf ("Error: can't allocate memory for a RX packet !!! \n"); return; } }   int main (void) { struct KprotoPckt KprotoPckt = { 1, 0, 128, {1234}, NULL }; f (&KprotoPckt);   free (KprotoPckt.Data), KprotoPckt.Data = NULL;   return 0; }

---

tu veux dire qu'il n'y a pas besoin de tester si c'est egal a null, c'est bien ca ?
je pensais qu'il fallait le faire, dans les doc il verifie toujours avant de poursuivre si le pointeur allouer est == a null.

ben en fait l'erreur, c'est une aerreur d'allocation memoir, donc en gros le realloc plante et mon programme plante.

apres verification datalength a l'air correct.

andromeda: Et si tu exécutes dans le debugger, n'as-tu pas plus d'information sur l'erreur ?
Un truc du genre "Damage after normal block blabla blabla" par exemple, indique qu'il y a eu débordement de tableau avant le realloc ou free...

Citation:

---

Envoyé par andromeda

ben en fait l'erreur, c'est une aerreur d'allocation memoir, donc en gros le realloc plante et mon programme plante.

apres verification datalength a l'air correct.

---

Tu n'as pas indiqué comment tu appelais la fonction. Je suppose que les données de la structure ne sont pas initialisées, d'où le crash avec realloc() à cause de la valeur indéterminée de data. Je rappelle que le premier paramètre de realloc() doit impérativement être :

• NULL ou
• une valeur retournée par un malloc(), realloc() ou calloc().

en reverifiant, la valeur de datalength est egal a 0, c'est peut etre ca.

met ca ne devrait pas planter je pense, car un realloc de 0 doit me garder l'allocation precedente.

je m'explique:

si j'avais allouer precedemment un tableu de 5 char par exemple avec malloc

j'ai tab[5]

si je realloue avec realloc et que la valeur de la reallocation est egal 0, je devrais toujours avoir mon tableau de 5 char ?

c'est une question , car j'ai des doute.

j'ai oublie de repondre a ton message precedent

au debut du programme j'initialise ma structure avec

Code:

---

1 2	memset(&KprotoPckt,0,sizeof(KprotoPckt));

---

je ne comprend pas tres bien ta question

Je rappelle que le premier paramètre de realloc() doit impérativement être :

* NULL
* une valeur retournée par un malloc(), realloc() ou calloc().

tu veux dire que je dois faire :

Code:

---

1 2 3 4 5 6 7

if (KprotoPckt->Data) KprotoPckt->Data = realloc (NULL, KprotoPckt->DataLength * sizeof (uint8_t *)); else KprotoPckt->Data = malloc (KprotoPckt->DataLength * sizeof (uint8_t *));

---

donc de remplacer KprotoPckt->Data par NULL dans le realloc ?

Merci de ton aide

Je pense que tu fais bien d'avoir des doutes, car à mon avis, realloc(..., 0) se comporte comme malloc(0) : Alloue un bloc de taille nulle...
Edit2: Ah non c'est encore pire: realloc(..., 0) libère la mémoire (du moins, c'est le cas sous VS2005).
Edit3: Et Visual suit le comportement standard.

Edit pour ta dernière question : Non. L'important est que le pointeur KprotoPckt->Data ne soit modifié que par un malloc() ou un realloc().

Et supprime ton memset(). Utilise une initialisation avec ={0} à la place.

Code:

---

1 2 3 4 5 6

// Au passage, donner le même nom à une structure est une variable, cay mal... struct KprotoPckt KprotoPckt = {0}   ...   + La fonction f() d'Emmanuel...

---

Citation:

---

Envoyé par Médinoc

Code:

---

1 2 3 4 5 6

// Au passage, donner le même nom à une structure est une variable, cay mal... struct KprotoPckt KprotoPckt = {0}   ...   + La fonction f() d'Emmanuel...

---

---

j'ai renome autrement ma fonction et initialiser correctement comme tu as dit plus haut.

donc en gros mon probleme viendrais de la valeur 0 du realloc ?
car si ca libere la memoire, en effet ca retourne null.

Merci.

en fait maintenant ca marche:

j'ai fait ceci:

Code:

---

1 2 3

KprotoPckt->Data = NULL; KprotoPckt->Data = malloc (KprotoPckt->DataLength * sizeof (uint8_t *));

---

car c'est appele dans une boucle.

concernant la valeur de retour 0 c'est normale pour mon traitement, je ne test donc pas la valeur NULL de l'allocation, comme ca la libere l'allocation memoire de toute facon quand c'est egal a zero.

maintenant tout marche bien.

Merci de m'avoir aider.