解决c++指针传递导致的重复释放问题

发布时间:2024-05-29 15:59:20 作者:yexindonglai@163.com 阅读(192)

first

先看这段代码,有多个地方都会释放 pPointer 的对象空间,但是为了防止重复释放,在第二次释放时做了判空处理;不为空才释放;

  1. #include <iostream>
  2. using namespace std;
  3. // 释放内存
  4. void release(int *pPointer){
  5. delete pPointer;
  6. pPointer = nullptr;
  7. }
  8. int main() {
  9. int *pPointer = new int();
  10. // 释放
  11. release(pPointer);
  12. if(pPointer) delete pPointer;
  13. return 0;
  14. }

执行结果如下, 报错了,告诉我执行了2次free函数,也就是重复释放了;这是怎么回事呢? 我明明有做判空啊;

  1. free(): double free detected in tcache 2
  2. Process finished with exit code 134

原因

原来;在调用release函数时,传入的指针变量,实际是经过了拷贝的;不信是吧,我们来做个实验,在main函数和release函数都打印下 pPointer 对象的地址,代码如下

  1. void release(int *pPointer){
  2. printf("release pPointer:%x\n",&pPointer); // 打印pPointer指针对象的地址
  3. delete pPointer;
  4. pPointer = nullptr;
  5. }
  6. int main() {
  7. int *pPointer = new int();
  8. printf("main pPointer:%x\n",&pPointer); // // 打印pPointer指针对象的地址
  9. // 释放
  10. release(pPointer);
  11. if(pPointer)delete pPointer;
  12. return 0;
  13. }

执行结果如下, 可以看到 2个指针地址是不同的,这也就证明了,在传参的过程中是发生了拷贝的;release 函数中的 pPointer 对象和main函数中的 pPointer 对象并不是同一个对象,只是他们都指向了同一块内存。

  1. main pPointer:f22d1fb0
  2. release pPointer:f22d1f98
  3. free(): double free detected in tcache 2
  4. Process finished with exit code 134

所以,他们的时间线如下:

  1. 在 release 函数内释放 pPointer, 并将 pPointer 赋值为 nullptr,此时,pPointer 所指向的内存块已经被释放,
  2. 但是只是将 release 函数内的 pPointer 变量赋值为 nullptr,main函数中 pPointer 还是指向原来的地址;因为指针指向的内存块已被释放,此时 main 函数中 pPointer 就是个野指针;
  3. 当指执行到 if(pPointer) delete pPointer; 代码时,pPointer 并不是空指针 nullptr ,它还指向原来的地址,所以判断if(pPointer)的结果为真(true),然后会执行 delete pPointer; ,但是原来地址的空间已经在release函数释放了,执行 delete pPointer; 后就会报错:double free detected in tcache 2

解决

方案一 :使用指针引用

将release函数的参数改为指针引用,如下,注意看,*后面多了个&符号,其他代码无需改动

  1. void release(int *& pPointer){
  2. delete pPointer;
  3. pPointer = nullptr;
  4. }

再次执行后,就不会报错了, 且main函数和release函数的 pPointer 对象地址也是相同的

  1. main pPointer:c12d0580
  2. release pPointer:c12d0580
  3. Process finished with exit code 0

方案二:使用二级指针

将release函数的参数改为二级指针,也能解决问题,如下

  1. void release(int **pPointer){
  2. printf("release pPointer:%x\n",pPointer);
  3. delete *pPointer; // 这里需要解引用
  4. *pPointer = nullptr; // 这里需要解引用
  5. }
  6. int main() {
  7. int *pPointer = new int();
  8. printf("main pPointer:%x\n",&pPointer);
  9. // 释放 - 需要加上取地址符号
  10. release(&pPointer);
  11. if(pPointer)delete pPointer;
  12. return 0;
  13. }

执行后结果如下

  1. main pPointer:d2699950
  2. release pPointer:d2699950
  3. Process finished with exit code 0

关键字c++