Discussion :

[Pikwik]

Pour avoir confirmation.

J'ai une DLL à chargement Dynamique, dans cette DLL j'utilise malloc pour allouer de la mémoire pour une chaine, et ma fonction me retourne un pointeur.

Arrétez moi si je me trompe, mais je ne peux pas faire de free(mon_pointeur) dans mon programme principal (car il ne connait pas la taille allouée ?), mais je peux faire une fonction, freeBuffer dans ma DLL qui s'occupe de le faire ?

Enfin voila, si vous pouvez m'éclairer.

[Médinoc]

En théorie, tu n'es pas censé pouvoir le faire et tu dois faire une fonction freeBuffer() dans la DLL.
En pratique, je sais que ça foirera en mode debug, pour le mode release je ne sais pas.

Sinon, tu peux aussi utiliser les fonctions d'allocation Win32, qui n'ont pas ce problème : LocalAlloc()/GlobalAlloc() (plus ou moins deprecated), HeapAlloc(), VirtualAlloc()...

[Aurelien.Regat-Barrel]

Tu peux mélanger les free/malloc entre ton exe et ta dll si ces 2 utilisent le même couple free/malloc, c.a.d la même CRT (elle aussi sour forme de dll utilisée par les 2 autres). Ca dépend des options de compilation, et c'est jouer avec le feu

[polonain2]

Par rapport à vos remarques, j'ai plusieurs questions:
- j'ai 2 dll, chargées par la même application (que j'appellerais dll1 et dll2). Dans dll1 j'ai une fonction qui fait un malloc sur une variable passé en paramètre de la fonction, dans dll2 j'ai une fonction qui fait un free sur cette même variable.

Code :

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

1
2
3
4
5
DLL1
void fct1 (StructPerso *part)
{...}
part->var = (char*)malloc(256*sizeof(char));
strcpy(part->var, "OK");

Code :

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

1
2
3
4
5
6
7
DLL2
void fct2
{...}
StructPerso part;
fct1(&part);
if(part.var != NULL) free(part.var);
part.var = NULL;

Ce code crash.
Si je met l'allocation dans la même dll (DLL2), aucun souci.
Je n'ai pas d'écrasement mémoire, j'ai vérifié.
D'après vos commentaires, ce problème est peut-être normal?
- si c'est le cas, est-ce que vous avez une référence (documentation web sur malloc/free ou autre) qui m'aiderai à argumenter (car il faut que j'explique pourquoi cette méthode d'allocation ne peut pas fonctionner)? j'ai beau chercher sur le net, je ne trouve rien sur malloc/free entre 2 dll.
- dernière question, il peut y avoir une différence de comportement entre Windows XP et Vista?
Merci d'avance.

[Médinoc]

En effet, ce genre de crash est normal.
La règle de base avec malloc/free, c'est "ce qui est alloué par un module doit être libéré par le même module".

Il faut donc que DLL1 expose une fonction spécifique pour libérer la mémoire qu'elle a allouée.

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void fct1 (StructPerso *part)
{
	...
	part->var = malloc(256*sizeof(*part->var));
	strcpy(part->var, "OK");
	...
}
 
void freefct1(char * p)
{
	free(p);
}

Code C :

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

1
2
3
4
5
6
7
8
9
void fct2
{
	...
	StructPerso part;
	fct1(&part);
	freefct1(part.var);
	part.var = NULL;
	...
}

Par contre, de là à trouver une référence officielle...
Tu peux chercher par ici:
C Run-Time Libraries
Allocating and freeing memory across module boundaries

PS: Le coup de l'utilisation de fonctions Windows comme LocalAlloc() etc. suit le même principe:

• LocalAlloc() et LocalFree() sont dans Kernel32.DLL, Ainsi que GlobalAlloc()/GlobalFree() et HeapAlloc()/HeapFree().
• CoTaskMemAlloc() et CoTaskMemFree() sont dans Ole32.DLL
• SysAllocString() et SysFreeString() sont dans OleAut32.DLL, ainsi que SafeArrayCreate() et SafeArrayDestroy().

[nicolas.sitbon]

La règle de base avec malloc/free, c'est "ce qui est alloué par un module doit être libéré par le même module".

Aurais tu une référence dans la norme appuyant tes propos ?

[nicolas.sitbon]

Code :

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

1
2
3
4
5
DLL1
void fct1 (StructPerso *part)
{...}
part->var = (char*)malloc(256*sizeof(char));
strcpy(part->var, "OK");

le cast du retour de malloc() est inutile sinon dangereux.
sizeof(char) renverra toujours 1.
je conseille :

Code :

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

part->var = malloc (256 * sizeof *part->var);

Code :

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

1
2
3
4
5
DLL2
void fct2
{...}
if(part.var != NULL) free(part.var);
part.var = NULL;

soit tu fais :

Code :

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

1
2
3
4
5
if (part.var != NULL) 
{
   free (part.var);
   part.var = NULL;
}

soit tu peux faire :

Code :

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

1
2
free (part.var);
part.var = NULL;

et généralement on faît même :

Code :

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

free (part.var), part.var = NULL;

en revanche pour ton soucis de malloc/free, je trouve bizarre ce comportement sous windows, dans la mesure ou les DLL partagent le même espace d'adressage, il ne devrait pas y avoir de problème. J'attends les explications d'un gourou...

[Aurelien.Regat-Barrel]

en revanche pour ton soucis de malloc/free, je trouve bizarre ce comportement sous windows, dans la mesure ou les DLL partagent le même espace d'adressage, il ne devrait pas y avoir de problème. J'attends les explications d'un gourou...

C'est comme si sous Linux chaque .so utilisait sa propre couple de fonctions malloc/free dans des implementation "maison"...

Ca ne pose pas de probleme avec les fonctions citees par Medinoc (LocalAlloc/LocalFree, etc...) parce que ces fonctions sont dans des dlls systemes utilisees par tous les processus. Mais malloc/free, c'est fourni par le compilateur... dans la fameuse "CRT". Le truc c'est qu'il est assez facile de se retrouver avec plusieurs CRT en memoire. Ca peut etre des CRT differentes ou plusieurs fois la meme CRT (le cas ici), ce qui ne change rien en fait.