[linux] probleme avec free()

Version imprimable

bonjour,

je travail sous eclipse j'ai un processeur 64 bits peut être le cause ?
j'ai juste fait un petit programme de test :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#include <stdio.h>
#include <stdlib.h>
 
int main (int argc, char** argv)
{
	int c=5;
	int * point =(int*) malloc( sizeof(int));
	printf("%d\n le c",c);
	point=&c;
	*point=6;
	printf("%d le point\n",c);
	free(point);
 
 
	return 0;}

le programme me retourne une 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
*** glibc detected *** /home/dev/workspace/pointuer/Debug/pointuer: free(): invalid pointer: 0xbf8ea88c ***
======= Backtrace: =========
/lib/libc.so.6[0x4d83adf1]
/lib/libc.so.6(cfree+0x90)[0x4d83e430]
/home/dev/workspace/pointuer/Debug/pointuer[0x804846d]
/lib/libc.so.6(__libc_start_main+0xe0)[0x4d7e8f70]
/home/dev/workspace/pointuer/Debug/pointuer[0x8048351]
======= Memory map: ========
08048000-08049000 r-xp 00000000 fd:00 10420814   /home/dev/workspace/pointuer/Debug/pointuer
08049000-0804a000 rwxp 00000000 fd:00 10420814   /home/dev/workspace/pointuer/Debug/pointuer
0892c000-0894d000 rwxp 0892c000 00:00 0 
4ce02000-4ce1d000 r-xp 00000000 fd:00 14712871   /lib/ld-2.6.so
4ce1d000-4ce1e000 r-xp 0001a000 fd:00 14712871   /lib/ld-2.6.so
4ce1e000-4ce1f000 rwxp 0001b000 fd:00 14712871   /lib/ld-2.6.so
4d7d3000-4d921000 r-xp 00000000 fd:00 14712873   /lib/libc-2.6.so
4d921000-4d923000 r-xp 0014e000 fd:00 14712873   /lib/libc-2.6.so
4d923000-4d924000 rwxp 00150000 fd:00 14712873   /lib/libc-2.6.so
4d924000-4d927000 rwxp 4d924000 00:00 0 
4e3ba000-4e3c5000 r-xp 00000000 fd:00 14713989   /lib/libgcc_s-4.1.2-20070503.so.1
4e3c5000-4e3c6000 rwxp 0000a000 fd:00 14713989   /lib/libgcc_s-4.1.2-20070503.so.1
b7d00000-b7d21000 rwxp b7d00000 00:00 0 
b7d21000-b7e00000 --xp b7d21000 00:00 0 
b7f07000-b7f08000 rwxp b7f07000 00:00 0 
b7f1c000-b7f1e000 rwxp b7f1c000 00:00 0 
b7f1e000-b7f1f000 r-xp b7f1e000 00:00 0          [vdso]
bf8d8000-bf8ee000 rw-p bf8d8000 00:00 0          [stack]

20/08/2007, 17h59
aoyou
Avec
Code:

1 2 3 4 printf("%d\n le c",c); /* point=&c; */ *point=6;
ça marche beaucoup mieux, mais ce n'est surement pas ce que tu veux faire.

Tu essaies de libèrer la mémoire allouée pour la variable c et non celle que tu as alloué toi-même. Ne t'étonnes-pas si tu as des problèmes...
20/08/2007, 18h01
Beldom

Cela vient de ton

Code:

point=&c;

Tu modifie la valeur du pointeur, pas de la valeur.
Donc tu as déplacé ton pointeur, le free essaie donc d'agir sur quelque chose qui n'a pas été alloué.

Normalement, si tu veux que point soit egal à l'adresse de c, tu dois le déclarer sans faire de malloc, puis dire point = &c.
Et donc pas besoin de free.

Si tu malloc, tu ne dois pas utiliser c, mais uniquement *point.

J'espère avoir été clair, sinon, n'hésite pas à demander plus de d'explication.
20/08/2007, 18h02
gl

Citation:

Envoyé par Asmod_D

int * point =(int*) malloc( sizeof(int));

Il est imperatif de tester le retour de malloc. Malloc peut echouer.
En outre le cast (int*) n'est pas utile et il peut etre plus avantageux de remplacer sizeof(int) par sizeof *point

Citation:

Envoyé par Asmod_D

point=&c;

L'adresse obtenue par malloc vient d'etre ecraser par l'adresse de la variable c. Tu ne pourras donc pas liberer la memoire alloue

Citation:

Envoyé par Asmod_D

free(point);

l'adresse contenue dans point n'etant pas une adresse fournie par malloc mais celle de la variable c, cette adresse n'est pas liberable ==> BUG

En outre apres avoir liberer de la memoire il est recommande de reinitialiser le pointeur a NULL, ce qui permet de tester la validite du pointeur dans le reste du code
20/08/2007, 19h21
AL1986

En faisant un malloc tu réserves une zone mémoire de la taille d'un int, or tu n'as pas du tout utiliser cette zone puisque tu n'as pas manipulé directement *point. En faisant point=&c tu à juste "déplacé" le pointeur étant donné qu'il ne pointera plus sur la zone allouée et qu'il pointera sur la même zone que c. Or &c n'a pas été allouée donc ne peut être libérée.

Citation:

Envoyé par gl

il peut etre plus avantageux de remplacer sizeof(int) par sizeof *point

Quels sont les avantages de sizeof(*point) par rapport à sizeof(int) ? Merci.

Citation:

Quels sont les avantages de sizeof(*point) par rapport à sizeof(int) ? Merci.
Code:

1 2 3 int * ptr =NULL; ptr = malloc(sizeof (*ptr)); if(ptr == NULL)exit(EXIT_FAILURE);
maintenant tu veux pas un int * mais un char * , tu n'as qu'a changer le type .
Code:

1 2 3 char * ptr =NULL; ptr = malloc(sizeof (*ptr));/* la taille du type pointé par ptr */ if(ptr == NULL)exit(EXIT_FAILURE);

20/08/2007, 23h03
Asmod_D

C vrai que j'ai quelques problemes avec les pointeurs

merci !