试图改写realloc函数在C

Question

作为惯例，我试图重写realloc函数

void updateSize(void* p,int length) 
{ 
    free(p); 
    malloc(sizeof(p) * length); 
} 

int main(int argc,char* argv[]) 
{ 
    int *y =malloc(sizeof(int)*3); 
    y = updateSize(y, 5); 
} 

但是当我尝试编译它，我得到以下错误：

void value not ignored as it ought to be. 

什么是这个错误的原因，我该如何解决它？

Answer 1

OP没有从函数返回值，也没有根据需要复制数据。

// Function return value of `void` needs to be `void *`. 
// `length` should be of type size_t 
// The former length of `p` needs to be passed. 
// Potential NULL pointers need testing. 
void updateSize(void* p,int length) { 
    // Missing data copy. 
    // Allocate new memory and copy before freeing old 
    free(p); 
    // Returned value from malloc needs saving. 
    // No reason for 'sizeof()' 
    malloc(sizeof(p) * length); 
} 

// Recommend 
void *updateSize2(void* p, size_t OldLength, size_t NewLength) { 
    void *p2 = malloc(NewLength); 
    if (p && p2) { 
    memcpy(p2, p, OldLength < NewLength ? OldLength : NewLength); 
    } 
    if (p2 || (NewLength == 0)) { 
    free(p); // Note A 
    } 
    return p2; 
} 

---

注答：有一个有趣的问题NewLength == 0时。某些malloc(0)实现返回NULL，其他返回指向“无数据”的指针。在前者中，NULL指针不总是总是意味着失败malloc()。现在@sharth正确地指出free(p)只能在内存分配失败时调用，这里使用的if()以NewLength为条件。