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都需要释放,不然就有内存泄漏。