问题描述

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

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

解决方案

代码顺序需要调整一下，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的内存块。

解决方案二：

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

解决方案三：

这样写会有内存泄漏，这里指释放了pOld，没有释放pNew，

然后就是如果要规范一点最好还是用指针，不要用整形来代替指针地址

int *pOld = (int *) malloc( sizeof(int) );
int *pNew = (int *) malloc( sizeof(int) );
//TODO something.
free(pNew);
free(pOld);

解决方案四：

pOld ，pNew都需要释放，不然就有内存泄漏。