c语言 - 为什么 malloc 根本不分配内存？

Question

我有以下结构：

struct block {
    void *addr; /*start address of memory for this block */
    int size;
    struct block *next;
};

我有以下代码来初始化每个块：

void block_init(struct block *b, void *addr, int size){
    /*Allocate space and fill b with the initial data.*/
    b = (struct block *)malloc(sizeof(struct block));
    if(b){
        b->addr = addr;
        b->size = size;
        b->next = NULL; 
    }    
}

我从另一个函数调用以下线路：

struct block *list;
block_init(freelist, mem, size);

但是，它从不初始化块。

我用gdb来测试这个，但是每次我得到一个空指针：

123     b = (struct block *)malloc(sizeof(struct block);
(gdb) next
124     if(b){
(gdb) print b
$2 = (struct block *) 0x0
(gdb) print b->size
Cannot access memory at address 0x8

我不知道发生了什么事，有人能帮我吗？

Answer 1

您已经使用了block *，所以如果您更改b的值，它将不会反映到调用函数。您应该使用block**。

void block_init(struct block **b /*HERE*/, void *addr, int size){
    /*Allocate space and fill b with the initial data.*/
    *b /*AND HERE*/ = (struct block *)malloc(sizeof(struct block));
    if(*b){
        (*b)->addr = addr;
        (*b)->size = size;
        (*b)->next = NULL; 
    }    
}

呼叫功能，

block_init(&list , mem, size);//pass by address

Answer 2

不反对@pranit kothari所说的，编写原始函数的一种可能更惯用的方式是

struct block* block_init(void *addr, int size) {
    /*Allocate space and fill b with the initial data.*/
    struct block* b = (struct block *)malloc(sizeof(struct block));
    if (b) {
        b->addr = addr;
        b->size = size;
        b->next = NULL; 
    }
    return b;
}

这样可以避免修改任何参数（我通常觉得这是一种"代码气味"），阅读起来更清晰，调用起来可能更整洁。

（顺便问一下，malloc参数应该是size * sizeof(struct block)吗？）