释放分配的内存

Question

gcc 4.4.5 c89

我有一个名为 create_object 的函数，我在其中为全局结构分配内存。我有一个名为destroy_object 的函数，我在其中检查指针是否为空，然后释放。以防我释放尚未分配的内存。但是，我已经通过连续两次调用destroy_object 对此进行了测试。但是，我在第二次调用时得到了堆栈转储。但是，我确信它不会释放，因为我已将指针分配给 NULL。所以它应该跳过免费功能。

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;
    }
}

非常感谢您的任何建议，

================== 编辑========== 基本上我在我的主函数中有一个对 create_object() 的调用，我做了一些处理，然后调用了 destory_object。

int main(void)
{
    create_object();

    /* Do some processing on the structure */

    destroy_object();

    return 0;
}

==========================最终编辑==== 静态结构 Config_t { 字符协议[LINE_SIZE]; 字符模式[LINE_SIZE]; } app_cfg[1] {{"", ""}};

现在我没有使用 malloc 和 free。

Answer 1

我只有一个建议。不要为此分配内存，这是浪费精力。

由于app_cfg 是一个文件级变量，因此一次只能有一个副本，因此分配和取消分配它没有什么意义。

只需将其创建为静态非指针并使用它：

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

您仍然可以提供create 和destroy 其中memset 的结构为零，但即使这样也可能不需要：

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

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

Answer 2

当我调用它两次时，在 Cygwin 下将此代码与 gcc 3.3.3 一起使用对我来说可以正常工作。你没有告诉我们你在这些功能之外做什么，所以先看看那里，例如也许您不小心在调用之间为 app_cfg 分配了一个垃圾非 NULL 值。此外，如果您没有使用“大牌”编译器，则有可能这是一个编译器错误（例如，它可能在编译时过于乐观，并假设您永远不会将 NULL 传递给 destroy_config）。尝试输入类似的内容：

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" );
        }
}

看看它是否真的“知道”它何时为空。