[optimisation] Gestion mémoire

Bonjour à tous,

Dans le cadre d'un projet OpenSource que je mène, j'ai été amené à réaliser une "sorte" d'ArrayList.
Cette structure me sert pour plusieurs traitement, dont un replace_all.
Le programme étant prévu pour fonctionner tout au long de la session utilisateur (comme un dock, conky, etc.), la gestion de la mémoire est très importante.

En passant le tout à valgrind, ce dernier me signal quelques fuites (je n'ai pas la sortie exacte sous la main pour le moment). C'est pourquoi je m'adresse à vous afin de voir si des améliorations sont possibles.

Pour information, pour ma structure, je me suis basé sur des double linked lists, et j'utilise la bibliothèque gtk, d'où les gchar, gboolean, etc.

Exemples d'exécution provoquant des fuite de mémoire :

Code:

---

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

int main ( int argc, char **argv ) { CidDataTable *test = cid_create_datatable(G_TYPE_STRING,"blah","blih","bloh","toto","tata","titi","tutu",G_TYPE_INT,5,12,G_TYPE_INVALID); printf("size: %d\n",cid_datatable_length(test)); cid_datatable_foreach(test,(CidDataAction)cid_datacase_print,NULL); test = cid_datatable_append(test,cid_datacontent_new_int(56)); cid_datatable_foreach(test,(CidDataAction)cid_datacase_print,NULL); printf("size: %d\n",cid_datatable_length(test)); test = cid_datatable_remove(test,cid_datacontent_new_string("toto")); cid_datatable_foreach(test,(CidDataAction)cid_datacase_print,NULL); printf("size: %d\n",cid_datatable_length(test)); cid_free_datatable(&test); printf("size: %d\n",cid_datatable_length(test));   return 0; }

---

Code:

---

1 2 3 4 5 6 7 8 9 10

int main ( int argc, char **argv ) { gchar *blah = "il etait une fois"; cid_str_replace_all(&blah," ","+"); g_print(":%s\n",blah);   return 0; }

---

La "bibliothèque" maintenant :

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 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377 378 379 380 381 382

typedef struct _CidDataTable CidDataTable; typedef struct _CidDataContent CidDataContent; typedef struct _CidDataCase CidDataCase;   typedef void (* CidDataAction) (CidDataCase *pCase, gpointer *pData);   struct _CidDataTable { size_t length; CidDataCase *head; CidDataCase *tail; };   struct _CidDataContent { union { gchar *string; gint iNumber; gboolean booleen; }; GType type; };   struct _CidDataCase { CidDataContent *content; CidDataCase *next; CidDataCase *prev; };   CidDataTable * cid_datatable_new (void) { CidDataTable *res = g_new0(CidDataTable,1); if (res != NULL) { res->length = 0; res->head = NULL; res->tail = NULL; } return res; }   CidDataCase * cid_datacase_new (void) { CidDataCase *ret = g_new0(CidDataCase,1); if (ret != NULL) { ret->content = NULL; ret->next = NULL; ret->prev = NULL; } return ret; }   /**  * Permet de creer un nouvel element pouvant etre insere dans une liste  * @param iType type de l'element  * @param value valeur de l'element  * @return nouvel element  */ CidDataContent * cid_datacontent_new (GType iType, void *value) { CidDataContent *ret = g_new0(CidDataContent,1); if (ret != NULL) { ret->type = iType; switch (iType) { case G_TYPE_STRING: ret->string = g_strdup((gchar *) value); break; case G_TYPE_INT: ret->iNumber = (gint) value; break; case G_TYPE_BOOLEAN: ret->booleen = (gboolean) value; break; default: return NULL; } } return ret; } #define cid_datacontent_new_string(val) cid_datacontent_new(G_TYPE_STRING, (void*)val) #define cid_datacontent_new_int(val) cid_datacontent_new(G_TYPE_INT, (void*)val) #define cid_datacontent_new_boolean(val) cid_datacontent_new(G_TYPE_BOOLEAN, (void*)val)     /**  * Permet de comparer deux elements  * @param d1 element 1  * @param d2 element 2  * @return TRUE si les elements sont egaux (en terme de valeur), sinon FALSE  */ gboolean cid_datacontent_equals (CidDataContent *d1, CidDataContent *d2) { if (d1 == NULL || d2 == NULL) return FALSE; if (d1->type != d2->type) return FALSE; switch (d1->type) { case G_TYPE_STRING: return g_strcmp0(d1->string,d2->string) == 0; case G_TYPE_INT: return d1->iNumber == d2->iNumber; case G_TYPE_BOOLEAN: return d1->booleen == d2->booleen; } }   /**  * Permet d'ajouter un element en fin de liste  * @param p_list liste a laquelle on souhaite ajouter un element  * @param data element a ajouter  * @return nouvelle liste  */ CidDataTable * cid_datatable_append(CidDataTable *p_list, CidDataContent *data) { if (p_list != NULL) { CidDataCase *p_new = cid_datacase_new(); if (p_new != NULL) { p_new->content = data; p_new->next = NULL; if (p_list->tail == NULL) { p_new->prev = NULL; p_list->head = p_new; p_list->tail = p_new; } else { p_list->tail->next = p_new; p_new->prev = p_list->tail; p_list->tail = p_new; } p_list->length++; } } return p_list; }   /**  * Permet de supprimer un element  * @param pCase element a supprimer  * @param pData donnees provenant du foreach  */ void cid_free_datacase_full (CidDataCase *pCase, gpointer *pData) { if (pCase != NULL) { if (pCase->content->type == G_TYPE_STRING) g_free(pCase->content->string); g_free(pCase->content); g_free(pCase); } } #define cid_free_datacase(val) cid_free_datacase_full(val, NULL)   /**  * Fonction permettant de liberer notre liste  * @param pointeur vers notre liste  */ void cid_free_datatable (CidDataTable **p_list) { if (*p_list != NULL) { cid_datatable_foreach(*p_list,(CidDataAction) cid_free_datacase_full, NULL); g_free(*p_list), *p_list = NULL; } }   /**  * Permet de parcourir tous les elements d'une liste donnee, et d'effectue le traitement voulu  * @param p_list liste que l'on souhaite parcourir  * @param func pointeur vers la fonction que l'on souhaite effectuer sur chaque element de la liste  * @param pData donnees a transmettre a la fonction func  * /!\ pData[0] contiendra toujours l'indice de l'element courant  */ void cid_datatable_foreach (CidDataTable *p_list, CidDataAction func, gpointer *pData) { if (p_list != NULL) { CidDataCase *p_temp = p_list->head; gint cpt = 1; while (p_temp != NULL) { if (pData == NULL) pData = g_new(gpointer,1); pData[0] = GINT_TO_POINTER(cpt); func (p_temp, pData); p_temp = p_temp->next; cpt++; } } }   /**  * Permet d'afficher la valeur d'un element  * @param pCase element dont on souhaite afficher la valeur  * @param pData donnees provenant eventuellement de cid_datatable_foreach  */ void cid_datacase_print (CidDataCase *pCase, gpointer *pData) { if (pCase != NULL) { switch (pCase->content->type) { case G_TYPE_STRING: g_print ("%s\n",pCase->content->string); break; case G_TYPE_INT: g_print ("%d\n",pCase->content->iNumber); break; case G_TYPE_BOOLEAN: g_print ("%s\n",pCase->content->booleen ? "TRUE" : "FALSE"); break; } } }   /**  * Fonction utilisee pour le str_replace_all  * @param pCase element courant de l'iteration  * @param pData donnees a utiliser  */ static void cid_datacase_replace (CidDataCase *pCase, gpointer *pData) { if (pCase != NULL) { gchar **c_tmp = pData[2]; if (GPOINTER_TO_INT(pData[0]) < GPOINTER_TO_INT(pData[1])) { g_sprintf(*c_tmp,"%s%s%s",*c_tmp,pCase->content->string,pData[3]); } else { g_sprintf(*c_tmp,"%s%s",*c_tmp,pCase->content->string); } } }   /**  * Permet de supprimer le premier element 'data' de la liste 'p_list'  * @param p_list liste de depart  * @param data element a supprimer  * @return nouvelle liste  */ CidDataTable * cid_datatable_remove(CidDataTable *p_list, CidDataContent *data) { if (p_list != NULL) { CidDataCase *p_temp = p_list->head; int found = 0; while (p_temp != NULL && !found) { if (cid_datacontent_equals(p_temp->content,data)) { if (p_temp->next == NULL) { p_list->tail = p_temp->prev; p_list->tail->next = NULL; } else if (p_temp->prev == NULL) { p_list->head = p_temp->next; p_list->head->prev = NULL; } else { p_temp->next->prev = p_temp->prev; p_temp->prev->next = p_temp->next; } cid_free_datacase(p_temp); p_list->length--; found = 1; } else { p_temp = p_temp->next; } } } return p_list; }   /**  * Permet de connaitre la taille d'une liste  * @param p_list liste dont on souhaite connaitre la taille  * @return taille de la liste  */ size_t cid_datatable_length(CidDataTable *p_list) { size_t ret = 0; if (p_list != NULL) { ret = p_list->length; } return ret; }   /**  * Fonction permettant de creer un tableau dynamique  * @param iDataType le type des donnees qui suivent  * @param ... les données, pouvant etre des chaines, des booleens, des  * entiers, ou des GType. Il faut obligatoirement terminer par un  * G_TYPE_INVALID  * @return pointeur vers notre structure  */ CidDataTable * cid_create_datatable (GType iDataType, ...) { CidDataTable *res = cid_datatable_new(); GType iCurrType = iDataType; va_list args; va_start(args,iDataType); void *current; while ((current = va_arg(args,void *)) != G_TYPE_INVALID) { CidDataContent *tmp = NULL; if ((GType) current == G_TYPE_BOOLEAN || (GType) current == G_TYPE_INT || (GType) current == G_TYPE_STRING) { iCurrType = (GType) current; continue; } switch (iCurrType) { case G_TYPE_BOOLEAN: tmp = cid_datacontent_new_boolean(current); break; case G_TYPE_STRING: tmp = cid_datacontent_new_string(current); break; case G_TYPE_INT: tmp = cid_datacontent_new_int(current); break; default: iCurrType = (GType) current; } res = cid_datatable_append(res,tmp); } va_end(args); return res; }   /**  * Permet de remplacer tous les motifs d'une chaine donnee  * @param string chaine que l'on souhaite modifier  * @param sFrom motif que l'on souhaite remplacer  * @param sTo motif avec lequel on remplace  */ void cid_str_replace_all (gchar **string, const gchar *sFrom, const gchar *sTo) { gchar **tmp = g_strsplit(*string,sFrom,0); CidDataTable *t_temp = cid_datatable_new(); while (*tmp != NULL) { t_temp = cid_datatable_append(t_temp,cid_datacontent_new_string(*tmp)); tmp++; } *string = (gchar *) malloc((strlen(*string)-(strlen(sFrom)*cid_datatable_length(t_temp))+(strlen(sTo)*cid_datatable_length(t_temp)))*sizeof(gchar)); g_sprintf(*string,""); gpointer *pData = g_new(gpointer,4); pData[0] = GINT_TO_POINTER(0); pData[1] = GINT_TO_POINTER(cid_datatable_length(t_temp)); pData[2] = string; pData[3] = (gchar *)sTo; cid_datatable_foreach(t_temp,(CidDataAction)cid_datacase_replace,pData); cid_free_datatable(&t_temp); }

---

Si vous avez besoin d'exécuter le code, je peux vous donner un lien pour récupérer les sources, puisque c'est OpenSource.

Merci d'avance.

Bonjour,

Je sais pas si je vais t'aider des masses mais se sera un debut.

Pour les fuites memoires tu as deja du t'assurer qu'il ne sagissait pas de fuites venant de la lib gtk! :D

Pour les options de valgrind, je le fais avec ca:
--leak-check=yes --show-reachable=yes

Et de facon generale, ca aide pas mal a voir ou sont les problemes.
Tu peux egalement envoyer tout ce que te sors valgrind dans un fichier, ca permet des fois un parcours plus simple.

Et quant a ton code, je sais pas trop quoi en dire :?

Je vois pas de malloc ou de chose comme ca, et tu appelles beaucoup de tes fonctions, donc pour te dire d'ou ca vient, c'est pas evident.
:(

Bref, j'espere quand meme t'avoir donne un debut de piste/reponse.

A bientot.

Bonjour,

Merci pour ta réponse. Effectivement, certaines fuites semblent venir de gtk... Cependant, mon code semble fuir également.

C'est vrai que je n'utilise pas ces options de valgrind, j'y penserai ;)

Pour ta remarque sur les malloc, en fait ils sont "encapsulés" dans les g_new.

Sinon, j'ai légèrement revu mon code et j'ai à priori résorbé quelques fuites, mais pas totalement.
Les modifs apportées sont du type :

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

/**  * Permet d'ajouter un element en fin de liste  * @param p_list liste a laquelle on souhaite ajouter un element  * @param data element a ajouter  * @return nouvelle liste  */ //CidDataTable * void cid_datatable_append(CidDataTable **p_list, CidDataContent *data) { if (*p_list != NULL) { CidDataCase *p_new = cid_datacase_new(); if (p_new != NULL) { p_new->content = data; p_new->next = NULL; if (*p_list->tail == NULL) { p_new->prev = NULL; (*p_list)->head = p_new; (*p_list)->tail = p_new; } else { (*p_list)->tail->next = p_new; p_new->prev = (*p_list)->tail; (*p_list)->tail = p_new; } (*p_list)->length++; } } //return p_list; }

---

Plutôt que de retourner une nouvelle liste, je modifie directement la liste concernée en passant la référence du pointeur à ma fonction.

J'ai modifié les fonctions suivantes de la sorte :

• cid_datatable_append
• cid_datatable_remove

Voyez-vous d'autres sources de fuites ?

Le main devient donc :

Code:

---

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

int main ( int argc, char **argv ) { CidDataTable *test = cid_create_datatable(G_TYPE_STRING,"blah","blih","bloh","toto","tata","titi","tutu",G_TYPE_INT,5,12,G_TYPE_INVALID); printf("size: %d\n",cid_datatable_length(test)); // affiche "size: 9" cid_datatable_foreach(test,(CidDataAction)cid_datacase_print,NULL); // affiche "blah blih bloh toto tata titi tutu 5 12" cid_datatable_append(&test,cid_datacontent_new_int(56)); cid_datatable_foreach(test,(CidDataAction)cid_datacase_print,NULL); // affiche "blah blih bloh toto tata titi tutu 5 12 56" printf("size: %d\n",cid_datatable_length(test)); // affiche "size: 10" cid_datatable_remove(&test,cid_datacontent_new_string("toto")); cid_datatable_foreach(test,(CidDataAction)cid_datacase_print,NULL); // affiche "blah blih bloh tata titi tutu 5 12 56" printf("size: %d\n",cid_datatable_length(test)); // affiche "size: 9" cid_free_datatable(&test); printf("size: %d\n",cid_datatable_length(test)); // affiche "size: 0"   return 0; }

---

Code:

---

cid_free_datacase_full (CidDataCase *pCase, gpointer *pData)

---

A quoi sert pData ? Il n'est pas utilisé dans le code de la fonction.


Plus problèmatique :

Code:

---

1 2 3 4 5 6 7 8 9

void cid_free_datatable (CidDataTable **p_list) { if (*p_list != NULL) { cid_datatable_foreach(*p_list,(CidDataAction) cid_free_datacase_full, NULL); g_free(*p_list), *p_list = NULL; } }

---

La fonction cid_datatable_foreach() sera appelée avec la fonction cid_free_datacase_full()
Dans le code

Code:

---

1 2 3 4 5 6 7 8

void cid_datatable_foreach (CidDataTable *p_list, CidDataAction func, gpointer *pData) { .... func (p_temp, pData); p_temp = p_temp->next; .... }

---

func() détruira *p_temp. Ce faisant, p_temp pointe alors sur un objet détruit et la ligne qui suit (p_temp = p_temp->next) est incorrecte.

Citation:

---

Envoyé par diogene

Code:

---

cid_free_datacase_full (CidDataCase *pCase, gpointer *pData)

---

A quoi sert pData ? Il n'est pas utilisé dans le code de la fonction.

---

C'est pour respecter le prototype d'encapsulation suivant :

Code:

---

1 2	typedef void (* CidDataAction) (CidDataCase *pCase, gpointer *pData);

---

Citation:

---

Envoyé par diogene

Plus problèmatique :

Code:

---

1 2 3 4 5 6 7 8 9

void cid_free_datatable (CidDataTable **p_list) { if (*p_list != NULL) { cid_datatable_foreach(*p_list,(CidDataAction) cid_free_datacase_full, NULL); g_free(*p_list), *p_list = NULL; } }

---

La fonction cid_datatable_foreach() sera appelée avec la fonction cid_free_datacase_full()
Dans le code

Code:

---

1 2 3 4 5 6 7 8

void cid_datatable_foreach (CidDataTable *p_list, CidDataAction func, gpointer *pData) { .... func (p_temp, pData); p_temp = p_temp->next; .... }

---

func() détruira *p_temp. Ce faisant, p_temp pointe alors sur un objet détruit et la ligne qui suit (p_temp = p_temp->next) est incorrecte.

---

Effectivement, valgrind me parlait d'"invalid read" qui peuvent provenir de là.
Je tâcherai de corriger cela.

Merci

Bonjour.

J'ai donc essayé de corriger ce que vous avez pointé et/ou ce que j'ai cru trouver "d'anormal" à l'aide de valgrind.

Les parties de code modifiées :

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

CidDataContent * cid_datacontent_new (GType iType, void *value) { CidDataContent *ret = g_new0(CidDataContent,1); if (ret != NULL) { ret->type = iType; switch (iType) { case G_TYPE_STRING: //ret->string = g_strdup((gchar *) value); ret->string = g_malloc0(strlen((gchar *) value)*sizeof(gchar)+1); strcpy(ret->string, (gchar *) value); break; case G_TYPE_INT: ret->iNumber = (gint) value; break; case G_TYPE_BOOLEAN: ret->booleen = (gboolean) value; break; default: return NULL; } } return ret; }

---

Code:

---

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

void cid_datatable_foreach (CidDataTable *p_list, CidDataAction func, gpointer *pData) { if (p_list != NULL) { CidDataCase *p_temp = p_list->head; gint cpt = 1; while (p_temp != NULL) { if (pData == NULL) pData = g_new(gpointer,1); pData[0] = GINT_TO_POINTER(cpt); CidDataCase *p_del = g_malloc0(sizeof(*p_temp)); memcpy(p_del,p_temp,sizeof(*p_temp)); func (p_temp, pData); p_temp = p_del->next; g_free(p_del); cpt++; } } }

---

Cependant, pour un main de ce type :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

int main (int argc, char **argv) { CidDataTable *test = cid_create_datatable(G_TYPE_STRING,"blah","blih","bloh","toto","tata","titi","tutu",G_TYPE_INT,5,12,G_TYPE_INVALID); printf("size: %d\n",cid_datatable_length(test)); // affiche "size: 9" cid_datatable_append(&test,cid_datacontent_new_int(56)); printf("size: %d\n",cid_datatable_length(test)); // affiche "size: 10" cid_datatable_remove(&test,cid_datacontent_new_string("toto")); printf("size: %d\n",cid_datatable_length(test)); // affiche "size: 9" cid_free_datatable(&test); printf("size: %d\n",cid_datatable_length(test)); // affiche "size: 0"   return 0; }

---

Cependant, un valgrind --leak-check=full ./cid me renvoie toujours ceci :

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

==30557== Memcheck, a memory error detector ==30557== Copyright (C) 2002-2009, and GNU GPL'd, by Julian Seward et al. ==30557== Using Valgrind-3.5.0-Debian and LibVEX; rerun with -h for copyright info ==30557== Command: ./cid ==30557== size: 9 size: 10 size: 9 size: 0 ==30557== ==30557== HEAP SUMMARY: ==30557== in use at exit: 137 bytes in 4 blocks ==30557== total heap usage: 75 allocs, 71 frees, 2,868 bytes allocated ==30557== ==30557== 4 bytes in 1 blocks are definitely lost in loss record 1 of 4 ==30557== at 0x4024C4C: malloc (vg_replace_malloc.c:195) ==30557== by 0x4950143: g_malloc (in /lib/libglib-2.0.so.0.2200.2) ==30557== by 0x8063583: cid_datatable_foreach (cid-utilities.c:551) ==30557== by 0x80635B7: cid_free_datatable (cid-utilities.c:536) ==30557== by 0x8052EC9: main (cid.c:255) ==30557== ==30557== 13 (8 direct, 5 indirect) bytes in 1 blocks are definitely lost in loss record 3 of 4 ==30557== at 0x4023F8B: calloc (vg_replace_malloc.c:418) ==30557== by 0x49500CB: g_malloc0 (in /lib/libglib-2.0.so.0.2200.2) ==30557== by 0x8063952: cid_datacontent_new (cid-utilities.c:381) ==30557== by 0x8052E9A: main (cid.c:251) ==30557== ==30557== 120 bytes in 1 blocks are definitely lost in loss record 4 of 4 ==30557== at 0x4024C4C: malloc (vg_replace_malloc.c:195) ==30557== by 0x4A61B17: getdelim (in /lib/i686/cmov/libc-2.9.so) ==30557== by 0x4D28D40: ??? (in /lib/libselinux.so.1) ==30557== by 0x4D31B8C: ??? (in /lib/libselinux.so.1) ==30557== by 0x4D21107: ??? (in /lib/libselinux.so.1) ==30557== by 0x400E283: ??? (in /lib/ld-2.9.so) ==30557== by 0x400E3B3: ??? (in /lib/ld-2.9.so) ==30557== by 0x400088E: ??? (in /lib/ld-2.9.so) ==30557== ==30557== LEAK SUMMARY: ==30557== definitely lost: 132 bytes in 3 blocks ==30557== indirectly lost: 5 bytes in 1 blocks ==30557== possibly lost: 0 bytes in 0 blocks ==30557== still reachable: 0 bytes in 0 blocks ==30557== suppressed: 0 bytes in 0 blocks ==30557== ==30557== For counts of detected and suppressed errors, rerun with: -v ==30557== ERROR SUMMARY: 3 errors from 3 contexts (suppressed: 135 from 6)

---

De ce que je comprends, les problèmes restants viennent de g_malloc0 et donc de la glib. Je ne peux donc plus "optimiser".
Vous confirmez ?

------------------------------

En fait, Je viens de déceler une nouvelle fuite.
Valgrind me retourne désormais ça :

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

==32315== Memcheck, a memory error detector ==32315== Copyright (C) 2002-2009, and GNU GPL'd, by Julian Seward et al. ==32315== Using Valgrind-3.5.0-Debian and LibVEX; rerun with -h for copyright info ==32315== Command: ./cid ==32315== size: 9 size: 10 size: 9 size: 0 ==32315== ==32315== HEAP SUMMARY: ==32315== in use at exit: 133 bytes in 3 blocks ==32315== total heap usage: 75 allocs, 72 frees, 2,868 bytes allocated ==32315== ==32315== 13 (8 direct, 5 indirect) bytes in 1 blocks are definitely lost in loss record 2 of 3 ==32315== at 0x4023F8B: calloc (vg_replace_malloc.c:418) ==32315== by 0x49500CB: g_malloc0 (in /lib/libglib-2.0.so.0.2200.2) ==32315== by 0x806398A: cid_datacontent_new (cid-utilities.c:381) ==32315== by 0x8052E9A: main (cid.c:251) ==32315== ==32315== 120 bytes in 1 blocks are definitely lost in loss record 3 of 3 ==32315== at 0x4024C4C: malloc (vg_replace_malloc.c:195) ==32315== by 0x4A61B17: getdelim (in /lib/i686/cmov/libc-2.9.so) ==32315== by 0x4D28D40: ??? (in /lib/libselinux.so.1) ==32315== by 0x4D31B8C: ??? (in /lib/libselinux.so.1) ==32315== by 0x4D21107: ??? (in /lib/libselinux.so.1) ==32315== by 0x400E283: ??? (in /lib/ld-2.9.so) ==32315== by 0x400E3B3: ??? (in /lib/ld-2.9.so) ==32315== by 0x400088E: ??? (in /lib/ld-2.9.so) ==32315== ==32315== LEAK SUMMARY: ==32315== definitely lost: 128 bytes in 2 blocks ==32315== indirectly lost: 5 bytes in 1 blocks ==32315== possibly lost: 0 bytes in 0 blocks ==32315== still reachable: 0 bytes in 0 blocks ==32315== suppressed: 0 bytes in 0 blocks ==32315== ==32315== For counts of detected and suppressed errors, rerun with: -v ==32315== ERROR SUMMARY: 2 errors from 2 contexts (suppressed: 135 from 6)

---

J'ai ajouté ceci :

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

void cid_datatable_foreach (CidDataTable *p_list, CidDataAction func, gpointer *pData) { if (p_list != NULL) { CidDataCase *p_temp = p_list->head; gboolean bCreateData = FALSE; if (pData == NULL) { pData = g_new(gpointer,1); bCreateData = TRUE; } gint cpt = 1; while (p_temp != NULL) { pData[0] = GINT_TO_POINTER(cpt); CidDataCase *p_del = g_malloc0(sizeof(*p_temp)); memcpy(p_del,p_temp,sizeof(*p_temp)); func (p_temp, pData); p_temp = p_del->next; g_free(p_del); cpt++; } if (bCreateData) { g_free(pData); } } }

---

La question que je me pause désormais, est de savoir si l'erreur relevée par valgrind dans cid_datacontent_new ne viendrait pas de ma structure en elle même ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11

typedef struct _CidDataContent CidDataContent;   struct _CidDataContent { union { gchar *string; gint iNumber; gboolean booleen; }; GType type; };

---

Et plus particulièrement de mon union "anonyme" ?