mémoire allouée libération

Question

gcc 4.4.5 C89

J'ai une fonction appelée create_object où j'allouer de la mémoire pour une structure globale. Et j'ai une fonction appelée destroy_object où je vérifie que le pointeur n'est pas nul, alors je suis libre. Juste au cas où je libère de la mémoire qui n'a pas été allouée. Cependant, j'ai testé cela en effectuant 2 appels consécutifs à destroy_object. Cependant, j'obtiens un vidage de pile au deuxième appel. Cependant, je suis sûr qu'il ne serait pas libre car j'ai assigné le pointeur à NULL. Donc, il devrait ignorer la fonction gratuite.

static struct Config_t { 
    char protocol[LINE_SIZE]; 
    char mode[LINE_SIZE]; 
} *app_cfg = NULL; 

int create_object() 
{ 
    app_cfg = malloc(sizeof *app_cfg); 
    memset(app_cfg, 0, sizeof *app_cfg); 
} 

void destroy_config() 
{ 
    /* Check to see if the memory is ok to free */ 
    if(app_cfg != NULL) { 
     free(app_cfg); 
     app_cfg = NULL; 
    } 
} 

Un grand merci pour toutes les suggestions,

================= ========== EDIT Basicially J'ai en Ma fonction principale est un appel à create_object() et je fais un traitement, puis je fais un appel à destory_object.

int main(void) 
{ 
    create_object(); 

    /* Do some processing on the structure */ 

    destroy_object(); 

    return 0; 
} 

========================= final Edit ==== statique struct {Config_t protocole char [LINE_SIZE]; mode char [LINE_SIZE]; } app_cfg [1] {{"", ""}};

Et maintenant je n'utilise pas malloc et gratuit.

Answer 1

Je n'ai qu'une suggestion. Ne pas allouer de la mémoire pour cela, c'est une perte d'effort. Puisque app_cfg est une variable de niveau fichier, vous ne pouvez en avoir qu'une copie à la fois, il est donc inutile de l'allouer et de la désallouer.

Il suffit de le créer comme non-pointeur et de l'utiliser statique:

static struct Config_t { 
    char protocol[LINE_SIZE]; 
    char mode[LINE_SIZE]; 
} app_cfg; 

Vous pouvez fournir encore un create et destroy qui memset la structure de zéros, mais même cela peut ne pas être nécessaire:

void create_object (void) { 
    memset(&app_cfg, 0, sizeof(app_cfg)); 
} 

void destroy_config (void) { 
    memset(&app_cfg, 0, sizeof(app_cfg)); 
} 

Answer 2

en utilisant ce code avec gcc 3.3.3 sous Cygwin fonctionne correctement pour moi quand je l'appelle deux fois. Vous ne nous avez pas dit ce que vous faites en dehors de ces fonctions, alors regardez d'abord, par exemple. peut-être que vous attribuez accidentellement une valeur non nulle NULL à app_cfg entre les appels. De plus, si vous n'utilisez pas un compilateur "big-name", il est possible que ce soit un bogue du compilateur (par exemple, il peut être trop optimiste au moment de la compilation et supposer que vous ne passerez jamais une valeur NULL à destroy_config). Essayez de mettre quelque chose comme:

void destroy_config() 
{ 

    /* Check to see if the memory is ok to free */ 
    if(app_cfg != NULL) { 
     printf("not null\n"); 
     free(app_cfg); 
     app_cfg = NULL; 
    }else{ 
     printf("null\n"); 
     } 
} 

pour voir si c'est vraiment "savoir" quand c'est nul.