Erreur avec la commande free

Version imprimable

Voilà je sèche sur un coredump généré par la commande free (fin du code ci-dessous). Si je le mets en commentaire, pas de problème. Sinon le programme crashe :aie:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
result = (char *)malloc(10*sizeof(char));
 if ( (result == NULL ) {
      // Not enough memory so stop execution and return failed
      strncpy(failureMessage, "Authentication failed. Not enought memory.\n", FAILURE_MSG_SIZE -1);
      failureMessage[FAILURE_MSG_SIZE] = (char)0;
      return QIPAUTHCALLOUT_FAIL;
   }
 
   result = getenv("AUTHCALLOUTDEBUG");
   if ( result != NULL ) {
      // check the value of result
      if ( strcmp(result,"ON") == 0 ) {
         // tracing is ON
         DEBUG = 1;
      }
   }
   /* Free up the memory space allocated to the result variable */
   free(result);

14/06/2006, 09h49
Médinoc

1°) Jamais de cast au retour de malloc() : Inutile en C, et pas de malloc() du tout en C++

2°)

Code:

result = getenv("AUTHCALLOUTDEBUG");

getenv() retourne l'adresse d'une chaîne statique, supposée invariable (son type de retour devrait être const)
Bref, pas de free() sur un retour de getenv().

3°) Au passage, la mémoire allouée par malloc() est leakée.

Bonjour,
Code:

1 2 result = (char *)malloc(10*sizeof(char));
- Pas de cast du malloc.
- sizeof(char) == 1 donc inutile
- Si tu as une taille fixe, pourquoi faire un malloc, une allocation statique suffit amplement.
Code:

1 2 3 4 5 6 7 if ( (result == NULL ) { // Not enough memory so stop execution and return failed strncpy(failureMessage, "Authentication failed. Not enought memory.\n", FAILURE_MSG_SIZE -1); failureMessage[FAILURE_MSG_SIZE] = (char)0; return QIPAUTHCALLOUT_FAIL; }
strncpy, si tu donnes une taille assez grande met le '\0' tout seul... Ce n'est pas la peine de le faire à la main... Autant utiliser strcpy pour ce que tu fais...

Citation:

result = getenv("AUTHCALLOUTDEBUG");

Depuis quand est-ce qu'on alloue une zone mémoire pour écraser le pointeur avec une autre adresse mémoire? Du coup, t'as perdu la première zone...
Code:

1 2 free(result);
Donc c'est logique que ton programme plante...
Jc

14/06/2006, 09h54
be_tnt

Merci Medinoc.

que veux tu dire par leakée?? :oops:
14/06/2006, 09h58
Médinoc

Citation:

Envoyé par fearyourself

strncpy, si tu donnes une taille assez grande met le '\0' tout seul... Ce n'est pas la peine de le faire à la main... Autant utiliser strcpy pour ce que tu fais...

strncpy() est faite pour marcher SANS qu'on donne une taille assez grande.
Et là, le zéro terminal dépend (hélas!) de l'implémentation : le standard exige peut-être le \0, mais en tout cas, le strncpy() de la CRT Microsoft ne le met pas.

Citation:

Envoyé par MSDN

If count is less than or equal to the length of strSource, a null character is not appended automatically to the copied string

Donc, si, le failureMessage[FAILURE_MSG_SIZE] = (char)0; doit rester (à supposer que failureMessage soit de taille FAILURE_MSG_SIZE+1, sinon, il y a un -1 à rajouter)
14/06/2006, 10h02
ankou82

Bonjour,

c'est dur de dire

Citation:

1°) Jamais de cast au retour de malloc() :

J'avais lu que l'on pouvait parfois caster, pour garder une compatibilité avec d'anciens compilateurs pré-ANSI ...
14/06/2006, 10h02
be_tnt
Citation:
Envoyé par fearyourself

Bonjour,

Code:

1 2 result = (char *)malloc(10*sizeof(char));

- Pas de cast du malloc.
- sizeof(char) == 1 donc inutile
- Si tu as une taille fixe, pourquoi faire un malloc, une allocation statique suffit amplement.
donc si je te suis bein, je devrais plutôt faire:

Code:

char result[10];
Citation:
Envoyé par fearyourself

Depuis quand est-ce qu'on alloue une zone mémoire pour écraser le pointeur avec une autre adresse mémoire? Du coup, t'as perdu la première zone...

Code:

1 2 free(result);

Donc c'est logique que ton programme plante...
hummm sorry je suis plutôt débutane en C avec un gros problème avec les pointeurs :aie:
14/06/2006, 10h07
fearyourself

Citation:

Envoyé par ankou82

Bonjour,

c'est dur de dire

J'avais lu que l'on pouvait parfois caster, pour garder une compatibilité avec d'anciens compilateurs pré-ANSI ...

Le jour où tu compileras avec un compilateur pré-ANSI, met des casts...

Citation:

donc si je te suis bien, je devrais plutôt faire:

Code:

char result[10];

Oui

Jc

Citation:

Envoyé par be_tnt

Merci Medinoc.

que veux tu dire par leakée?? :oops:

leak = fuite
memory leak = fuite mémoire (mémoire non libérée)

Voic le rapport de mon gestionnaire de mémoire après exécution de 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
33
34
35
36
37
38
39
40
41
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define FAILURE_MSG_SIZE 128
 
enum
{
   OK,
   QIPAUTHCALLOUT_FAIL,
   dummy
};
 
int main()
{
   int DEBUG = 0;
   char failureMessage[FAILURE_MSG_SIZE];
   char *result = (char *)malloc(10 * sizeof(char));
   if ( result == NULL )
   {
      // Not enough memory so stop execution and return failed
      strncpy(failureMessage, "Authentication failed. Not enought memory.\n", FAILURE_MSG_SIZE - 1);
      failureMessage[FAILURE_MSG_SIZE] = (char)0;
      return QIPAUTHCALLOUT_FAIL;
   }
 
   result = getenv("AUTHCALLOUTDEBUG");
   if ( result != NULL )
   {
      // check the value of result
      if ( strcmp(result, "ON") == 0 )
      {
         // tracing is ON
         DEBUG = 1;
      }
   }
   /* Free up the memory space allocated to the result variable */
   free(result);
   return 0;
}

Code:

1
2
3
4
5
6
7
8
9
10
11
 
FRMWRK.DBG_SYSALLOC=1
SYSALLOC Overload (85 rec)
SYSALLOC Successful initialization: 85 records available
SYSALLOC min=4294967295 max=4294967295 delta=0
SYSALLOC Err: Not-matched list:
SYSALLOC Bloc 003D24F0 (10 bytes) malloc'ed at line 18 of 'main.c' not freed
SYSALLOC Released Memory
FRMWRK.Termine
 
Press ENTER to continue.

En fait ton code est beucoup trop compliqué. Ceci suffit :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main()
{
   int DEBUG = 0;
 
   char *result = getenv("AUTHCALLOUTDEBUG");
   if ( result != NULL )
   {
      // check the value of result
      if ( strcmp(result, "ON") == 0 )
      {
         // tracing is ON
         DEBUG = 1;
      }
   }
   return 0;
}

14/06/2006, 10h10
be_tnt

huh du coup:

Code:

result = getenv(MYVAR);

ne marche plus (error: incompatible types in assignment)
14/06/2006, 10h14
be_tnt

euh tu peux expliquer l'output de ta commande?
14/06/2006, 10h19
Emmanuel Delahaye

Citation:

Envoyé par be_tnt

euh tu peux expliquer l'output de ta commande?

La ligne importante est celle-ci :

Code:

SYSALLOC Bloc 003D24F0 (10 bytes) malloc'ed at line 18 of 'main.c' not freed

Faut que je traduise ?
14/06/2006, 10h22
be_tnt

Citation:

Envoyé par Emmanuel Delahaye

La ligne importante est celle-ci :

Code:

SYSALLOC Bloc 003D24F0 (10 bytes) malloc'ed at line 18 of 'main.c' not freed

Faut que je traduise ?

Donc si je comprend bien, la mémoire réservée par le malloc exécuté en line 18 n'a pas été libérée. Je croyais que quand on utilisait des pointeurs, il fallait faire un malloc et donc ensuite un free :?
14/06/2006, 10h22
fearyourself
Citation:

Envoyé par be_tnt

huh du coup:

Code:

result = getenv(MYVAR);

ne marche plus (error: incompatible types in assignment)

Logique, je disais que pour une allocation de taille fixe, une allocation statique était plus adaptée. Dans ton problème aucune allocation est nécessaire. Seul un pointeur l'est.
Code:

1 2 3 char *result; result = getenv("AUTHCALLOUTDEBUG");
Jc
14/06/2006, 10h33
Emmanuel Delahaye
Citation:

Envoyé par be_tnt

Donc si je comprend bien, la mémoire réservée par le malloc exécuté en line 18 n'a pas été libérée. Je croyais que quand on utilisait des pointeurs, il fallait faire un malloc et donc ensuite un free :?

Comme tu as remplacé la valeur du pointeur par une autre,
Code:

1 2 3 4 char *result = (char *)malloc(10 * sizeof(char)); <...> result = getenv("AUTHCALLOUTDEBUG");
la valeur originale est perdue et le free() ne libère pas la bonne zone (comportement indéfini).

Mon gestionnaire d'allocations mémoire constate que le bloc n'a pas été libéré, c'est tout. Il n'en donne pas les raisons.

En fait, comme déjà expliqué, il n'y a pas besoin de malloc(), à moins qu'on veuille conserver la valeur... Dans ce cas, strdup() est certainement plus adapté. cette fonction n'est pas standard C, mais POSIX.1, donc très portable. Elle est facile à implémenter si besoin est...

Et je rappelle qu'une affectation de chaine se fait avec strcpy() en C...
14/06/2006, 10h47
be_tnt
si je veux ré-utiliser un pointeur, est-ce que je dois faire quelque chose?
Code:

1 2 3 4 5 6 7 8 char *value; value = getenv(VAR1); [....] value = getenv(VAR2);
14/06/2006, 11h12
fearyourself
Citation:
Envoyé par be_tnt

si je veux ré-utiliser un pointeur, est-ce que je dois faire quelque chose?

Code:

1 2 3 4 5 6 7 8 char *value; value = getenv(VAR1); [....] value = getenv(VAR2);
Non, c'est très bien comme ça,
Jc
14/06/2006, 11h20
be_tnt

:merci: