Discussion :

[mymimi]

Bonjour à tous
Je veut dé-allouer un tableau 2 dimensions (nombre de colonnes différent du nombre de lignes) alloué dynamiquement.

Allocation tableau:

Code :

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

1
2
3
p=(double**) malloc (sizeof(double*)*(H));
		for(i=0;i<n;i++)
			p[i]=(double*) malloc (sizeof(double)*(n));

Libération mémoire:

Code :

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

1
2
3
4
5
6
7
for(i=0;i<H;i++)
		{
			free(p[i]);
			p[i] = NULL ;
		}
		free(p);
		p= NULL;

Un message d'erreur s'affiche:

Debug Error
Program ..... .exe
Heap corruption detected: after normal block (#62) at 0x00435EF0.
CRT detected that the application wrote to memory after end of heap buffer.

Est ce qu'il y a quelqu'un qui peut m'aider à bien libérer la mémoire?
Merci d'avance

[Obsidian]

Je pense que ton problème ne vient pas de ces deux extraits de code mais de ce qu'il y a entre les deux, et qui doit corrompre la mémoire avant que tu la libères.

En particulier, les lignes 2 et 3 me laissent perplexe :

Code :

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

1
2
3
…
    for(i=0;i<n;i++)
    p[i]=(double*) malloc (sizeof(double)*(n));

Je ne pense pas que ce soit réellement ce que tu cherches à faire.

[moins1]

Il me semble que tu devrais boucler avec H et non avec n pour allouer la deuxième dimension de ton tableau puisque que tu as créé un tableau de H pointeur de double.

[Médinoc]

Autant d'allocations, c'est compliqué et ça nuit à la localité mémoire. Normalement il y a moyen de se contenter d'une allocation par dimension.

[mymimi]

Merci pour tous

Il me semble que tu devrais boucler avec H et non avec n pour allouer la deuxième dimension de ton tableau puisque que tu as créé un tableau de H pointeur de double.

Je veut que le le tableau contient H colonnes et n lignes. Comment je peut l'allouer donc?

Je pense que ton problème ne vient pas de ces deux extraits de code mais de ce qu'il y a entre les deux, et qui doit corrompre la mémoire avant que tu la libères.

J'ai vérifié par des printf, tout va bien sauf que le programme se plante au niveau de libération de la mémoire (précisément ce tableau). Que doit je faire?

[Metalman]

Toujours en version "simple" (déconseillée par Médinoc car multiples mallocs peu utiles (dans le cas des char* à taille variable c'est une bonne idée, mais pas si on veut une matrice)) :

L'important dans ces algos est de BIEN garder l'ordre des colonnes et des lignes...
Personnellement j'itère sur les lignes pour allouer les colonnes.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#define COL nb_colonnes_a_indiquer
#define LN nb_lignes_a_indiquer
int N;
double **mat;
 
mat = malloc(LN * sizeof (double*));
for (N = 0; N != LN; N++)
{
  mat[N] = malloc(COL * sizeof (double));
}

Et à free :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
#define COL nb_colonnes_a_indiquer
#define LN nb_lignes_a_indiquer
int N;
double **mat;
 
for (N = 0; N != LN; N++)
{
  free(mat[N]);
}
free(mat);

Si tu fais un unique malloc 2 mallocs en suivant la méthode de Médinoc, un unique free 2 frees seront nécessaires tout à la fin, et évitera de la fragmentation mémoire !

EDIT : en règle générale... garde en tête : 1 malloc => 1 free !
Certaines fonctions standards ne mallocent pas, d'autres oui... donc bien lire la doc !

EDIT2 : comme dis plus bas : Médinoc fait 2 mallocs, pardon
Donc 2 frees !