Discussion :

[Asmod_D]

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 :

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

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 :

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

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]

[aoyou]

Avec

Code :

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

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...

[Beldom]

Cela vient de ton

Code :

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

	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.

[gl]

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

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

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

[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.

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

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

[scriptoff]

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

Code :

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

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 :

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

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

[Asmod_D]

C vrai que j'ai quelques problemes avec les pointeurs

merci !