Surcharge des fonctions malloc et free

Bonjour,

Afin de détecter les fuites de mémoire, j'ai surchargé les fonctions malloc et free. Pas de soucis à ce niveau.
Maintenant j'ai plusieurs programmes avec les sources que je veux tester.
Pour cela je dois manuellement ajouter deux fichiers (un .c et un.h) ainsi que les 3 directives suivantes (tout en haut de chaque fichier .c) :

Code:

---

1 2 3

#include "checkMemory.h" #define malloc(sz) check_malloc((sz), __FILE__, __LINE__) #define free(sz) check_free((sz), __FILE__, __LINE__)

---

Je cherche à savoir si il est possible d'ajouter ces trois directives automatiquement à chacun de mes fichiers sources (dans un projet, j'ai une dizaines de fichiers sources).

J'utilise mingw32 comme compilateur sous windows 7.

Merci beaucoup

Bonjour

Pourquoi ne pas avoir ajouté les #define sur malloc et free dans checkMemory.h ?
Sinon pour rajouter la ligne #include autant éditer les 10 fichiers à la main c'est presque plus rapide que de le faire avec un script du genre

Code:

---

1 2 3 4 5 6 7 8 9

cat << EOF > header #include "memoryCheck.h" #define ... #define ... EOF for file in la liste des fichiers sources sans doute un *.c; do cp $file $file.bkp cat header $file.bkp > $file done

---

C'est quick and dirty pas à l'épreuve de tous les cas de figure etc ...
Il doit y avoir un équivalent en powershell pas trop compliqué à construire je suppose, avec des foreach et des add-content, mais je ne suis pas un pro.

Sinon ... pourquoi ne pas utiliser valgrind ?

Bonjour,

Je ne peux pas mettre les deux defines dans checkmemory.h car dans checkmemory.c j'appelle les vraies fonctions malloc et free donc je ne dois pas les overloader...

(sauf si j'ai raté quelque chose...)

Et sinon pour le #include, oui faire un script j'y ai pensé mais je voulais savoir s'il n'y avait pas une autre méthode.

Et pour valgrind, c'est super cool comme outil mais ne marche que sous Linux et je travaille sous Windows ;)

mingw32 + W7, pas sûr que valgrind fonctionne dessus...
Tu peux utiliser Dr.Memory dessus, c'est ce que je fais et le rendu est similaire...

L'appel à Dr.Memory se fait exactement comme Valgrind :

Code:

---

1 2	gcc -g -G3 -ggdb drmemory -show_reachable -leak_check

---

(au lieu de --show-reachable=yes --leak-check=full)

Mais je ne peux que plussoyer kwariz : un coup de valgrind sur un linux sera toujours très utile !
J'ai eu des comportements très différents entre mingw32, cygwin et linux !
Mingw32 segfault beaucoup moins que Linux ou Cygwin, et tolère des dépassements légers de la mémoire....

Citation:

---

Envoyé par Aspic

Bonjour,

Je ne peux pas mettre les deux defines dans checkmemory.h car dans checkmemory.c j'appelle les vraies fonctions malloc et free donc je ne dois pas les overloader...

(sauf si j'ai raté quelque chose...)

---

Bah si tu les mets tout à la fin il n'y aura pas de problème. Un #define n'a d'effets qu'après sa définition jamais avant, c'est juste une directive du préprocesseur.

Ah oui .... pas de valgrind sous win32 ... je ne connais pas l'outil proposé par metalman mais bon il fleure bon :)

Citation:

---

Envoyé par kwariz

Bah si tu les mets tout à la fin il n'y aura pas de problème. Un #define n'a d'effets qu'après sa définition jamais avant, c'est juste une directive du préprocesseur.

---

Tu as raison, je suis bête c'est l'habitude de mettre les defines dans le .h j'ai oublié qu'on pouvais aussi les mettre dans les fichiers sources :mrgreen:

Merci pour ton aide, pour le include qui reste, je le ferais en script ;)

EDIT : mince ca marche pas à la fin des fichiers .c les #define ne sont pas pris en compte...
Et si je les mets à la fin du .h alors les fonctions malloc et free du fichier .c sont overloadées et donc je suis en boucle infinie :(

Mets tout dans le header et pour forcer l'appel à la fonction malloc plutôt qu'à la macro il te suffit de mettre malloc entre parenthèse :

Code:

---

1 2 3 4 5

void* xmalloc(size_t size, size_t line, const char* source) { fprintf(stderr, "malloc size : %lu in %s line %lu\n", size, source, line); return (malloc)(size); }

---

Je peux pas tester maintenant mais merci beaucoup pour l'astuce, je ne connaissait pas du tout ;)