Discussion :

[Tex-Twil]

aidez moi svp car avec ce code, quand je libere la memoire de char *final j'ai une erreur "*** glibc detected *** free(): invalid next size (fast): 0x0804b5e0 ***"

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
 
	unsigned char *keyAlice = NULL;
	unsigned char *final = NULL;
        ...
	// Computing secret shared keys with other peers's public key
	//Alice
	keyAlice = (unsigned char *)malloc(DH_size(dhAlice));
	if(keyAlice == NULL) { perror("malloc"); exit(2); }
 
	final = (unsigned char *)malloc(DH_size(dhAlice));	
	if(final == NULL) { perror("malloc"); exit(2); } 	
 
	ret = DH_compute_key(keyAlice, dhBob->pub_key, dhAlice);
	if( ret == -1 ) { perror("Dh Compute"); exit(2); }
 
	printf("Alice calculated : ");
	for (j=0; j<ret; j++)
	{
	// decallage du pointeur
		sprintf(final+(2*j), "%02X",keyAlice[j]);
	}
	printf("%s\n", final); 
 
	if(final != NULL) free(final);
	if(keyAlice != NULL) free(keyAlice);

[PRomu@ld]

Code :

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

1
2
3
 
keyAlice = (unsigned char *)malloc(DH_size(dhAlice)); 
   if(keyAlice == NULL) { perror("malloc"); exit(2);

Ne cast plus le retour de malloc, c'est quasiment inutile (sauf si tu as un très vieux compilateur)

Quel est le code associé à 2 dans les codes d'erreur ? (utilise EXIT_SUCCESS, EXIT_FAILURE) Utilise un code propre à toi mais dans ce cas, ça fais le dans un define que tu réutilises plus tard.

Code :

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

1
2
3
 
if(final != NULL) free(final); 
   if(keyAlice != NULL) free(keyAlice);

Attention, il faut mettre les pointeurs à NULL après un free :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
if(final != NULL) 
{
     free(final);
     final = NULL;
} 
if(keyAlice != NULL) 
{
     free(keyAlice); 
     keyAlice = NULL;
}

Ensuite, pour en revenir à ta question, le problème arrive généralement lorsque tu fais un "double free", chose qui arrive si tu ne fais pas ce que je viens de te montrer après les free.

[Tex-Twil]

j'ai fait les modification mais c'est toujours pareil. Le prog plante qund j'essaye de liberer final

[Tex-Twil]

Je crois aue j'ai trouve.

Je n'allouais pas la bonne taille a la chaine final. Elle doit etre deux fois plus grande que la chaine dhAlice.

[fearyourself]

Par contre, ce code:

Code :

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

1
2
3
4
5
6
7
8
9
10
if(final != NULL)
{
     free(final);
     final = NULL;
}
if(keyAlice != NULL)
{
     free(keyAlice);
     keyAlice = NULL;
}

est trop compliqué. Par définition, free ne fait rien si le pointeur passé en argument est NULL, donc c'est inutile de le tester.

Ce code est plus simple et tout aussi juste (et écrit d'une façon qui fera plaisir à quelqu'un!):

Code :

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

1
2
3
 
free(final) , final = NULL;
free(keyAlice) , keyAlice = NULL;

Jc

[PRomu@ld]

est trop compliqué

je n'ai fais que recopier le code déjà fourni et rajouté les affectations à NULL ...

Par définition, free ne fait rien si le pointeur passé en argument est NULL, donc c'est inutile de le tester.

J'en suis bien conscient :
http://rperrot.developpez.com/articl...llocationC/#L3

[fearyourself]

est trop compliqué

je n'ai fais que recopier le code déjà fourni et rajouté les affectations à NULL ...

Par définition, free ne fait rien si le pointeur passé en argument est NULL, donc c'est inutile de le tester.

J'en suis bien conscient :
http://rperrot.developpez.com/articl...llocationC/#L3

Je le sais, la remarque n'était pas dirigé vers toi personnellement. Le code était, certes, le tien mais c'est celui qui m'est tombé sous la main pour la remarque.



Jc