重新分配从函数返回的值会产生段错误

Question

我有一个 C 语言的 LDA 代码，我正在尝试调试它，并且我已经绞尽脑汁有一段时间了。

lda_model *model = NULL;
model = malloc(sizeof(lda_model));
model = quiet_new_lda_model(corpus->num_terms, NTOPICS);
printf ("%f\n", model->alpha); // Segfaults here

如果我们查看模型创建函数，

lda_model* quiet_new_lda_model(int num_terms, int num_topics) {
    int i;
    lda_model* model;

    model = malloc(sizeof(lda_model));
    model->num_topics = num_topics;
    model->num_terms = num_terms;
    model->alpha = 1.0;
    printf ("%f\n", model->alpha); // Prints 1.0
    model->log_prob_w = malloc(sizeof(double*)*num_topics);
    for (i = 0; i < num_topics; i++)
    {
        model->log_prob_w[i] = malloc(sizeof(double)*num_terms);
    memset(model->log_prob_w[i],0,sizeof(double)*num_terms);
    }
    printf ("%f\n", model->alpha); // Prints 1.0
    return(model);
}

quiet_new_lda_model 函数和原始调用者之间的事务可能存在什么问题？

谢谢！

Answer 1

好吧，首先，调用者中的 malloc 是不必要的，并且会导致内存泄漏。尝试将这四行替换为

lda_model *model = quiet_new_lda_model(corpus->num_terms, NTOPICS);
printf ("%f\n", model->alpha); // Segfaults here

但这不太可能是段错误的原因。接下来我要问的是，quiet_new_lda_model 的原型在调用者中是否可见？如果不是，您的指针可能会被截断为 int。您可以通过将所有三行 printf 行更改为读取来找出答案。

printf("%p\n", model);

如果第三行 printf 打印的值与前两行不同，则问题可能是缺少原型。

编辑：判断您是否缺少原型的另一种方法是查找如下警告消息：

test.c:4: warning: initialization makes pointer from integer without a cast

根据上下文和您使用的编译器，措辞可能会有所不同。如果您使用的是 GCC，请使用 -Wall 命令行开关来获得更强的提示：

test.c:4: warning: implicit declaration of function ‘quiet_new_lda_model’
test.c:4: warning: initialization makes pointer from integer without a cast

“隐式声明”意味着“您没有告诉我有关此函数的任何信息，因此我将假设它接受任意数量的参数并返回int。”正如您可以想象的那样，这几乎总是错误的。

（作为一般规则，如果您使用 GCC，则命令行上应该始终有 -Wall。）

Answer 2

这不是解决方案，但你有内存泄漏；您不应该在第一个代码块中分配模型，因为它是在函数中分配的。

我怀疑真正的根本原因是@Zack 所建议的 - 你没有原型，因此返回的地址被截断（我假设你使用的是 64 位系统）。

使用。

lda_model* quiet_new_lda_model(int, int); 

在第一个代码块的顶部

Answer 3

首先，在quiet_new_lda_model函数中执行malloc时存在内存泄漏，而没有释放之前声明的资源。您不需要在调用者中执行 malloc。其次，lda_model 在哪里/如何定义？头文件是否包含在调用该函数的文件中？

Answer 4

您最近是否更改了 lda_model 的定义？更改后您是否重新编译了所有使用它的源文件？

Answer 5

修复代码中明显的错误。您有内存泄漏需要修复。第二，再次调试代码，在更高的函数中执行类似这样的代码 if(model) printf("%f", model->alfa);应该可以帮助你前进。它无法预测，但有时，堆栈断言可能会产生段错误。

更新：尝试使用“%ld”而不是“%f”进行更改。也许会有帮助