free

Version imprimable

29/08/2008, 09h27
ninours23

calloc / realloc / free
Bonjour, j'ai un pb et je ne vois pas ou

je calloc une matrice :
Code:

1 2 3 4 5 6 7 8 double **mat = (double**)calloc(20, sizeof(double*)); for (i = 0; i < 20; i++){ mat[i] = (double*)calloc(3, sizeof(double)); }
en cours de programme je la realloc :
Code:

1 2 mat = (double**)realloc(mat, 10*sizeof(double*));
j'ai aussi essayé de la realloc comme ca
Code:

1 2 3 4 5 for (i = 10; i < 20; i++){ free(mat[i]); } mat = (double**)realloc(mat, 10*sizeof(double*));
puis en fin de programme je la free :
Code:

1 2 3 4 5 6 7 8 for (i = 0; i < 10; i++){ free(mat[i]); } free(mat);
Le probleme c'est qu'un coup l'execution se passe bien et de temps en temps ca plante, si je commente le free ca passe tout le temps
Le free m'a l'air bien, je pense que je realloc mal, mais je ne vois pas l'erreur
please help me :cry:
29/08/2008, 09h43
BainE

bonjour,

quelques remarques qui peut etre te feront avancer

1°) on peux faire un realloc de taille plus petite que la taille d origine ?
- est ce utile ? (on peux pas remettre a zero et utiliser que les 10 premiers indices)
- que deviennent les pointeur dont on perd l adresse (les 10 derniers ici) ?
- soit, passons, apres ton realloc alloues tu bien des nouveaux pointeurs sur double comme fait avec les calloc ?
29/08/2008, 10h19
ninours23

Citation:

Envoyé par BainE

bonjour,

quelques remarques qui peut etre te feront avancer

1°) on peux faire un realloc de taille plus petite que la taille d origine ?
- est ce utile ? (on peux pas remettre a zero et utiliser que les 10 premiers indices)
- que deviennent les pointeur dont on perd l adresse (les 10 derniers ici) ?
- soit, passons, apres ton realloc alloues tu bien des nouveaux pointeurs sur double comme fait avec les calloc ?

Je te remercie de ces quelques reflexions mais je n'avance pas sur mon probleme :cry:

bon et en essayant ca ?

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
 
double **mat = (double**)calloc(20, sizeof(double*));
 
for (i = 0; i < 20; i++){ 
    mat[i] = (double*)calloc(3, sizeof(double));
}
 
/* traitement puis liberation */
 
for (i = 0; i < 20; i++){ 
    free(mat[i]);
}
free(mat);
mat = NULL;
 
/* reallocation */
mat = (double**)calloc(10, sizeof(double*));
 
for (i = 0; i < 10; i++){ 
    mat[i] = (double*)calloc(3, sizeof(double));
}
 
/* re traitement puis re liberation */
 
for (i = 0; i < 10; i++){ 
    free(mat[i]);
}
free(mat);
mat = NULL;

29/08/2008, 15h05
stephl

Citation:

Envoyé par ninours23

Le probleme c'est qu'un coup l'execution se passe bien et de temps en temps ca plante, si je commente le free ca passe tout le temps
Le free m'a l'air bien, je pense que je realloc mal, mais je ne vois pas l'erreur
please help me :cry:

Pourrais-tu poster le reste de ton code?

Citation:

Envoyé par BainE

bon et en essayant ca ?

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
 
double **mat = (double**)calloc(20, sizeof(double*));
 
for (i = 0; i < 20; i++){ 
    mat[i] = (double*)calloc(3, sizeof(double));
}
 
/* traitement puis liberation */
 
for (i = 0; i < 20; i++){ 
    free(mat[i]);
}
free(mat);
mat = NULL;
 
/* reallocation */
mat = (double**)calloc(10, sizeof(double*));
 
for (i = 0; i < 10; i++){ 
    mat[i] = (double*)calloc(3, sizeof(double));
}
 
/* re traitement puis re liberation */
 
for (i = 0; i < 10; i++){ 
    free(mat[i]);
}
free(mat);
mat = NULL;

j'ai pas essaye mais ton "/* traitement puis liberation * /" vide entierment mat alors que moi j'ai une mat de 20 lignes, je veux garder les 10 premieres et vider la mémoire de 10 a 20

29/08/2008, 15h25
stephl

Citation:

Envoyé par ninours23

j'ai pas essaye mais ton "/* traitement puis liberation * /" vide entierment mat alors que moi j'ai une mat de 20 lignes, je veux garder les 10 premieres et vider la mémoire de 10 a 20

Pourrais-tu montrer le reste de ton code? Il se pourrait que le problème soit ailleurs...
29/08/2008, 15h34
ninours23

Citation:

Envoyé par stephl

Pourrais-tu montrer le reste de ton code? Il se pourrait que le problème soit ailleurs...

Y'a rien de plus
Si vous essayez le code de mon premier message ca ne fonctionne pas
29/08/2008, 15h35
BainE

Ben copie les 10 premiers elem dans un tableau nouvellement cree pour l occasion, puis libere le gros.

Ou bien garde ton tableau de 20, mais n utilise que les 10 premiers et tu liberes qu a la fin.

Parce que un realloc plus petit que la taille original j ai aucune idée du comportement et pas le temps de chercher dans la norme.
29/08/2008, 15h37
stephl

Citation:

Envoyé par ninours23

Y'a rien de plus
Si vous essayez le code de mon premier message ca ne fonctionne pas

Bien montre le fichier EXACT que tu compiles parce que personnellement, ça fonctionne chez moi (même si je trouve que ce n'est pas forcément une bonne pratique).
29/08/2008, 18h11
diogene
Code:

1 2 3 4 for (i = 10; i < 20; i++){ free(mat[i]); } mat = (double**)realloc(mat, 10*sizeof(double*));
Est correct si on suppose que la reallocation ne peut échouer (accessoirement, le cast (double**) est inutile).

Toutefois, la valeur de mat peut changer (rien n'assure que dans le cas d'une réduction de la taille mémoire demandée, la valeur de mat soit conservée, bien que cela semble logique).
Si c'est le cas et que ce code est dans une fonction avec mat comme paramètre, la valeur de mat dans le programme appelant cette fonction est elle inchangée et pointe maitenant sur une zone désallouée. Ce qui va faire planter dans ce cas le free(mat).
29/08/2008, 18h38
stephl

Citation:

Envoyé par diogene

Toutefois, la valeur de mat peut changer (rien n'assure que dans le cas d'une réduction de la taille mémoire demandée, la valeur de mat soit conservée, bien que cela semble logique).
Si c'est le cas et que ce code est dans une fonction avec mat comme paramètre, la valeur de mat dans le programme appelant cette fonction est elle inchangée et pointe maitenant sur une zone désallouée. Ce qui va faire planter dans ce cas le free(mat).

Très juste, c'est pour cela qu'il nous faut le code complet qui pose problème.
29/08/2008, 19h20
Vincent Rogier

Ne jamais affecter la valeur de retour d'un realloc() au pointeur qui a été passé à realloc().

Car si realloc() retourne NULL et que tu ne possèdes pas d'autres pointeurs sur le bloc mémoire en question, tu ne pourra plus libérer le bloc mémoire et donc tu as une fuite !

Donc toujours affecter le retour de realloc() à un pointeur intermédiaire afin de réaffecter à sa valeur au pointeur d'origine si besoin ...
02/09/2008, 15h03
3DArchi

Le realloc vers plus petit n'a d'intérêt que si tu as besoin de récupérer de la mémoire. Sinon conserve sa taille originale. Ca sera la garantie contre moins de bug. Il faut aussi faire attention à libérer les éléments qui seront perdus (du 10ème au 20ème si j'ai bien compris). Et effectivement, passer par une variable intermédiaire pour gérer le NULL (car tu perds non seulement ton premier pointeur mais aussi les 10/20 autres contenus).
11/09/2008, 09h59
gangsoleil

Bonjour,

J'ai aussi l'impression que tu oublies un peu de tester le code de retour de tes fonctions d'allocation (et pas seulement de realloc, pour laquelle il est nécessaire de passer par une seconde variable).

Peux-tu poster un code minimal, compilable, reproduisant l'erreur ?

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 #include <stdlib.h> #include <assert.h> #include <stdio.h> #define TOREALLOC 10 int main (void) { double **mat = calloc(20, sizeof(double*)); unsigned i; for (i = 0; i < 20; i++){ mat[i] = calloc(3, sizeof(double)); assert(mat[i] != NULL); } printf("\nBefore: %p \n", mat); mat = realloc(mat, TOREALLOC * sizeof(double*)); assert(mat != NULL); printf("\nafter: %p - sizeof double: %d \n", mat, sizeof(double)); for (i = 10; i < 20; i++){ printf("\nValue: %d - %p \n", i, mat[i]); fflush(stdout); free(mat[i]); } return 0; }
J'ai rien inventé, c'est ton code de manière un peu plus verbeux. Change la valeur de TOREALLOC, regarde ce qu'il se passe et conclus.

combien de mémoire est-ce que tu veux allouer ?
combien en alloues-tu réélement ?
pourquoi est-ce qu'on a ces valeurs farfelues ?
pourquoi est-ce qu'on a pas besoin de caster lorsqu'on realloc/calloc ?
Si tu changes ton double en int, est-ce que ça marche ? pourquoi ?

+

Citation:

Envoyé par ninours23

Bonjour, j'ai un pb et je ne vois pas ou

en cours de programme je la realloc :

Code:

1
2
 
mat = (double**)realloc(mat, 10*sizeof(double*));

OK. Le code ayant l'air correct, je mets en œuvre mon contrôle de mémoire :

http://emmanuel-delahaye.developpez.com/clib.htm
Module SYSALLOC

Pas bon :

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
 
Usage xxx[ /T][ /E][ /O][ <options application>]
FRMWRK.DBG_SYSALLOC=1
SYSALLOC Overload (1365 rec)
SYSALLOC Successful initialization: 1365 records available
done in 0 s
SYSALLOC min=4294967295 max=4294967295 delta=0
SYSALLOC Err: Not-matched list:
SYSALLOC Bloc 00261128 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Bloc 00261148 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Bloc 00261168 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Bloc 00261188 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Bloc 002611A8 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Bloc 002611C8 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Bloc 002611E8 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Bloc 00261208 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Bloc 002807E0 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Bloc 00280800 (24 bytes) calloc'ed at line 18 of 'C:\dev\mem\main.c' no
t freed
SYSALLOC Released Memory
FRMWRK.Execution terminated
 
Process returned 0 (0x0)   execution time : 0.036 s
Press any key to continue.

Citation:

j'ai aussi essayé de la realloc comme ca

Code:

1
2
3
4
5
 
for (i = 10; i < 20; i++){
free(mat[i]);
}
mat = (double**)realloc(mat, 10*sizeof(double*));

C'est mieux :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Usage xxx[ /T][ /E][ /O][ <options application>]
FRMWRK.DBG_SYSALLOC=1
SYSALLOC Overload (1365 rec)
SYSALLOC Successful initialization: 1365 records available
done in 0 s
SYSALLOC min=4294967295 max=4294967295 delta=0
SYSALLOC All-matched
SYSALLOC Released Memory
FRMWRK.Execution terminated
 
Process returned 0 (0x0)   execution time : 0.051 s
Press any key to continue.

Citation:

Le probleme c'est qu'un coup l'execution se passe bien et de temps en temps ca plante, si je commente le free ca passe tout le temps
Le free m'a l'air bien, je pense que je realloc mal, mais je ne vois pas l'erreur
please help me :cry:

Essayons en mode debug :

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
 
Usage xxx[ /T][ /E][ /O][ <options application>]
FRMWRK.DBG_SYSALLOC=1
SYSALLOC Overload (1365 rec)
SYSALLOC Successful initialization: 1365 records available
SYSALLOC 'M 002A0F10     16 4294967295   88:C:\clib\ed\src\chro.c'
SYSALLOC 'C 002A0FA8     80 4294967295   13:C:\dev\mem\main.c'
SYSALLOC 'C 002A1000     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1020     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1040     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1060     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1080     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A10A0     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A10C0     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A10E0     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1100     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1120     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1140     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1160     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1180     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A11A0     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A11C0     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A11E0     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1200     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002A1220     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002C07E0     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'C 002C0800     24 4294967295   18:C:\dev\mem\main.c'
SYSALLOC 'F 002A1140      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002A1140 matched
SYSALLOC 'F 002A1160      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002A1160 matched
SYSALLOC 'F 002A1180      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002A1180 matched
SYSALLOC 'F 002A11A0      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002A11A0 matched
SYSALLOC 'F 002A11C0      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002A11C0 matched
SYSALLOC 'F 002A11E0      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002A11E0 matched
SYSALLOC 'F 002A1200      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002A1200 matched
SYSALLOC 'F 002A1220      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002A1220 matched
SYSALLOC 'F 002C07E0      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002C07E0 matched
SYSALLOC 'F 002C0800      0 4294967295   29:C:\dev\mem\main.c'
SYSALLOC 002C0800 matched
SYSALLOC 'F 002A0FA8     40 4294967295   31:C:\dev\mem\main.c'
SYSALLOC 002A0FA8 matched
SYSALLOC 'R 002A0FA8     40 4294967295   31:C:\dev\mem\main.c'
SYSALLOC 'F 002A1000      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A1000 matched
SYSALLOC 'F 002A1020      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A1020 matched
SYSALLOC 'F 002A1040      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A1040 matched
SYSALLOC 'F 002A1060      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A1060 matched
SYSALLOC 'F 002A1080      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A1080 matched
SYSALLOC 'F 002A10A0      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A10A0 matched
SYSALLOC 'F 002A10C0      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A10C0 matched
SYSALLOC 'F 002A10E0      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A10E0 matched
SYSALLOC 'F 002A1100      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A1100 matched
SYSALLOC 'F 002A1120      0 4294967295   38:C:\dev\mem\main.c'
SYSALLOC 002A1120 matched
SYSALLOC 'F 002A0FA8      0 4294967295   42:C:\dev\mem\main.c'
SYSALLOC 002A0FA8 matched
done in 0.065 s
SYSALLOC 'F 002A0F10      0 4294967295  114:C:\clib\ed\src\chro.c'
SYSALLOC 002A0F10 matched
SYSALLOC min=4294967295 max=4294967295 delta=0
SYSALLOC All-matched
SYSALLOC Released Memory
FRMWRK.Execution terminated
 
Process returned 0 (0x0)   execution time : 0.099 s
Press any key to continue.

Est-ce que les tailles allouées sont correctes ?

Apparement, oui, comme le confirme les résultats sur ce code :

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
 
#include <stdlib.h>
 
int main (void)
{
   int i;
 
   double **mat = calloc (20, sizeof *mat);
 
   for (i = 0; i < 20; i++)
   {
      mat[i] = calloc (3, sizeof *mat[i]);
   }
 
#if 0
   mat = realloc (mat, 10 * sizeof *mat);
#else
   for (i = 10; i < 20; i++)
   {
      free (mat[i]);
   }
   mat = realloc (mat, 10 * sizeof *mat);
#endif
 
   for (i = 0; i < 10; i++)
   {
      free (mat[i]);
   }
   free (mat);
 
   return 0;
}

Le problème est donc ailleurs.