C语言内存管理问题，要注重细节

Question

int pOld = (int) malloc( sizeof(int) );
int pNew = (int) malloc( sizeof(int) );
pOld=pNew;
free(pOld);
这样写代码，有什么问题？

Answer 1

代码顺序需要调整一下，pOld在赋值成pNew之前就必须free掉，不然第一次malloc出来的内存块就没有任何指针指向它了，也就是说我们失去了回收它的途径，所以正确的代码应该像下面这样：

int pOld = (int) malloc( sizeof(int));
int pNew = (int) malloc( sizeof(int));
free(pOld);
pOld=pNew;

如果像之前的代码：

pOld=pNew;
free(pOld);

这样free掉的其实是新申请的那块内存，因为pOld此时已经被赋值，指向了第二次malloc的内存块。