为什么malloc在这里返回NULL?

时间:2016-03-15 18:28:49

标签: c pointers null malloc

所以我的程序总是返回一个分段错误,但我无法理解为什么这么试图用GDB进行调试并且它向我展示了这个:

(gdb) backtrace
#0  0x001a98ef in _int_malloc (av=0x2d8440, bytes=8) at malloc.c:3835
#1  0x001abedc in __GI___libc_malloc (bytes=8) at malloc.c:2924
#2  0x0804cd6a in init_capsula (item1_=2, item2_=2)
    at src/modulos/modulos_auxiliares/capsula/capsula.c:25
#3  0x0804d366 in total_dados_produto (f=0x8055838, filial=0x0, mes=6, 
    cod=0xbffff23c "AF1184") at src/modulos/faturacao/faturacao.c:208
#4  0x0804b237 in queries (q=3, c1=0x0, c2=0x0, f=0x8055838, v=0x0) at src/interface.c:815
#5  0x0804b6f4 in menu (c1=0x8055008, c2=0x8055420, f=0x8055838, v=0x0) at src/interface.c:976
#6  0x080487ad in main () at src/interface.c:1037

然后我确定了来自第2帧的问题的根源,因此决定检查出来并获得以下输出:

(gdb) frame 2
#2  0x0804cd6a in init_capsula (item1_=2, item2_=2)
    at src/modulos/modulos_auxiliares/capsula/capsula.c:25
25          c->item1 = (int*) malloc((sizeof (int))*item1_);

它告诉我malloc正在返回一个NULL,但是我看不到这行的问题,所有内容都已经完全初始化,因为我确认了下一步:

(gdb) print ((sizeof (int))*item1_)
$1 = 8

为什么malloc不能分配这么小的空间?我是不是真的傻了?

我将把函数init_capsula放在这里(malloc所在的那个)给你们看:

Capsula init_capsula(int item1_, int item2_){
     Capsula c = (Capsula) malloc (sizeof (struct capsula));

     c->tipo  = -1;

     if (item1_ > 0)
         c->item1 = (int*) malloc((sizeof (int))*item1_); /*Problematic line*/
     else c->item1 = NULL;

     if (item2_ > 0)
         c->item2 = (float*) malloc((sizeof (float))*item2_);
     else c->item2 = NULL;

     c->q1 = 0;
     c->q2 = 0;

     return c;
}

Capsula是指向如下定义的结构的指针:

struct capsula{
    int tipo;

    int     q1;
    int *item1;

    int       q2;
    float *item2;
 };

编辑:

如果我尝试使用以下命令运行valgrind:

     valgrind --tool=memcheck --leak-check=full make run

它输出这个,我觉得没有用。

    make: *** [run] Segmentation fault (core dumped)
    ==5848== 
    ==5848== HEAP SUMMARY:
    ==5848==     in use at exit: 62,771 bytes in 1,819 blocks
    ==5848==   total heap usage: 6,060 allocs, 4,241 frees, 580,609 bytes allocated
    ==5848== 
    ==5848== LEAK SUMMARY:
    ==5848==    definitely lost: 0 bytes in 0 blocks
    ==5848==    indirectly lost: 0 bytes in 0 blocks
    ==5848==      possibly lost: 0 bytes in 0 blocks
    ==5848==    still reachable: 62,771 bytes in 1,819 blocks
    ==5848==         suppressed: 0 bytes in 0 blocks
    ==5848== Reachable blocks (those to which a pointer was found) are not shown.
    ==5848== To see them, rerun with: --leak-check=full --show-reachable=yes
    ==5848== 
    ==5848== For counts of detected and suppressed errors, rerun with: -v
    ==5848== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

EDIT2:

我最终通过正确使用valgrind来理解这个问题,因为我在make上使用它时,我应该在程序本身上使用它(如注释中所示)。问题是在一个非常不同的地方,在一个我忘记写一个malloc的地方,感谢所有帮助的人,现在我终于明白我应该如何使用valgrind

3 个答案:

答案 0 :(得分:2)

如果调试器显示您在此行上触发了分段错误:

c->item1 = (int*) malloc((sizeof (int))*item1_);

这可能意味着两件事:

  • c是一个错误的指针,可能是NULL,但前面的语句c->typo = -1;也应该失败。

  • 竞技场可能已损坏,问题出在之前执行的代码中。

答案 1 :(得分:1)

我正在回答我自己的问题,因为正如我在上次编辑时所说,我在评论的帮助下找到了答案。

我最终通过正确使用valgrind来理解这个问题,因为我在make上使用它时,我应该在程序本身上使用它(如注释中所示)。问题是在一个非常不同的地方,在一个我忘记写一个malloc的地方,所以没有必要详细了解,感谢所有帮助的人,现在我终于明白我应该如何使用valgrind

答案 2 :(得分:0)

我认为您的问题与您使用malloc的方式有关。 Malloc分配一块内存块,将指针返回到块的开头。你应该写:

Capsula * c = (Capsula *) malloc (sizeof (struct capsula));

实际上,除非c是指向结构的指针,否则编写 c-> tipo = -1; 是非法的。例如, - > c-> tipo 中的运算符是 *(c).tipo 的快捷方式。