-1

我正在调整对象的大小和数组。我做了一个临时对象，但是当我不删除它时，Valgrind 显示内存泄漏和错误。删除它会导致段错误。只是想知道 Valgrind 在抱怨什么……

void Obj::resize()
{
  Obj *temp = new Obj[size * 2];   //<-- line 92
  for (int i = 0; i < size; i++)
    temp[i] = objarray[i];
  delete [] objarray;
  objarray = temp;
  size *= 2;
  //delete temp;   //<-- causes segfault
  //delete [] temp;  // also segfaults, tried them both just in case :\
}

这是 Valgrind 的报告：

==9292== HEAP SUMMARY:
==9292==     in use at exit: 21,484 bytes in 799 blocks
==9292==   total heap usage: 3,528 allocs, 2,729 frees, 91,789 bytes allocated
==9292==
==9292== 21,484 (2,644 direct, 18,840 indirect) bytes in 1 blocks are definitely lost in loss record 4 of 4
==9292==    at 0x4008409: operator new[](unsigned int) (vg_replace_malloc.c:357)
==9292==    by 0x804AC7E: MyClass::resize() (file.cpp:92)
==9292==    by 0x804AC34: MyClass::add(int, int) (file.cpp:82)
==9292==    by 0x804AAE6: getline(std::istream&, MyClass&) (file.cpp:66)
==9292==    by 0x8049772: main (otherfile.cpp:39)
==9292==
==9292== LEAK SUMMARY:
==9292==    definitely lost: 2,644 bytes in 1 blocks
==9292==    indirectly lost: 18,840 bytes in 798 blocks
==9292==      possibly lost: 0 bytes in 0 blocks
==9292==    still reachable: 0 bytes in 0 blocks
==9292==         suppressed: 0 bytes in 0 blocks
==9292==
==9292== For counts of detected and suppressed errors, rerun with: -v
==9292== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

我对gdb不太好，但得到了这个回溯：

    (gdb) run
Starting program: 

Program received signal SIGSEGV, Segmentation fault.
0x46ed40e3 in free () from /lib/libc.so.6
Missing separate debuginfos, use: debuginfo-install glibc-2.15-58.fc17.i686 libgcc-4.7.2-2.fc17.i686 libstdc++-4.7.2-2.fc17.i686
(gdb) backtrace
#0  0x46ed40e3 in free () from /lib/libc.so.6
#1  0x4742dba0 in operator delete(void*) () from /lib/libstdc++.so.6
#2  0x0804ad68 in MyClass::resize (this=0xbffff28c) at file.cpp:98
#3  0x0804ac35 in MyClass::add (this=0xbffff28c, month=10, day=31)
    at file.cpp:82
#4  0x0804aae7 in getline (input=..., a=...) at file.cpp:66
#5  0x08049773 in main (argc=1, argv=0xbffff344) at otherfile.cpp:39
(gdb)

我认为删除它是不好的，因为它应该只是让指针悬空，所以我得到一个段错误并不让我感到惊讶。不过，为什么它会导致内存问题？任何想法将不胜感激。

4

3 回答 3

1

事实上，你不能在那里删除它，因为你已经将它分配给objarray以后使用。

最有可能的是，您没有objarray在析构函数中删除；或其他一些功能正在重新分配它而不首先删除旧数组。

我会使用std::vector而不是手工制作的数组来为我处理释放。

于 2013-05-13T17:14:17.963 回答
0

只有在非常低级的库代码中才需要显式delete（或）。delete[]在其他任何地方，您都应该使用智能指针。

这是一种更好的方法，（IMO）更容易理解：

std::unique_ptr<Obj[]> temp(new Obj[size * 2]);

// copy stuff from objarray to temp

swap(objarray, temp);

就是这样。如果交换成功，unique_ptr析构函数将释放旧缓冲区。objarray如果在复制过程中抛出异常，它将释放新的临时缓冲区。在任何一种情况下objarray（也应该是std::unique_ptr<Obj[]>）都会留下一个有效的缓冲区。

于 2013-05-13T17:13:21.393 回答
0

如果这是您调用的第二次（或更晚）时间resize，那么这很可能是一种情况，因为您正在尝试delete在一个已经有 a 的堆上，double delete因为它temp已经被删除了，现在您所有的objArray写入都变成了一个块属于堆管理的内存，而不是你的代码。

这里也可能发生各种其他潜在问题，例如内存现在正用于某个 OTHER 对象，并且它正在将内容写入堆内存，然后您将其用作 objArray。

你不应该删除temp你试图这样做的地方。只是不要。

于 2013-05-13T17:19:50.740 回答