我的代码在删除此代码时崩溃

Question

尝试删除它时出现分段错误。

我知道你想删除这个，但它是我的前任留下的。我知道我应该采取的一些预防措施 ，这些都已经过验证并得到处理。

我不知道什么样的情况可能会导致这次崩溃，只是偶尔发生一次。大约 95% 的情况下，代码运行得很好，但有时似乎会以某种方式损坏并崩溃。

顺便说一句，该类的析构函数不执行任何操作。

我是否应该假设有什么东西正在其他地方破坏我的堆并且 this 指针以某种方式搞砸了？

编辑：根据要求，崩溃代码：

long CImageBuffer::Release()
{
  long nRefCount = InterlockedDecrement(&m_nRefCount);
  if(nRefCount == 0)
  {
    delete this;
  }
  return nRefCount;
}

该对象已使用新创建，它不在任何类型的数组中。

Answer 1

最明显的答案是：不要删除它。

如果您坚持这样做，请使用常见的方法来查找错误：
1.使用valgrind（或类似工具）查找内存访问问题
2.编写单元测试
3.使用调试器（准备好长时间盯着屏幕 - 取决于您的项目有多大）

Answer 2

看来您的 new 和 delete 不匹配。请注意，delete this; 只能用于使用 new 分配的对象（并且在覆盖 operator new 的情况下，或多个副本C++ 运行时的特定 new 与当前作用域中找到的 delete 相匹配）

Answer 3

释放时的崩溃可能会很痛苦：这种情况不应该发生，而当它发生时，代码太复杂而无法轻松找到解决方案。

注意：使用 InterlockedDecrement 让我假设您正在 Windows 上工作。

记录一切

我自己的解决方案是大量记录构造/破坏，因为调试时崩溃很可能永远不会发生：

1. 记录构造，包括 this 指针值和其他相关数据
2. 记录破坏，包括this 指针值和其他相关数据

这样，您将能够查看 this 是否被释放两次，甚至根本没有分配。

...一切，包括堆栈

我的问题发生在托管 C++/.NET 代码中，这意味着我可以轻松访问堆栈，这是一件幸事。您似乎在使用纯 C++，因此检索堆栈可能是一件苦差事，但它仍然非常有用。

您应该尝试从互联网加载代码来打印每个日志的当前堆栈。我记得玩过 http://www.codeproject.com/KB/threads/StackWalker。 aspx 为此。

请注意，您需要处于调试版本，或者将 PDB 文件与可执行文件一起使用，以确保堆栈被完全打印。

...一切，包括多次崩溃

我相信你在 Windows 上：你可以尝试捕获 SEH 异常。这样，如果发生多次崩溃，您将看到全部崩溃，而不是只看到第一个，并且每次您都可以在日志中标记“正常”或“崩溃”。我什至使用映射来记住分配/释放的地址，从而组织日志以将它们显示在一起（而不是按顺序）。

我在家，所以无法向您提供确切的代码，但是在这里，Google 是您的朋友，但要记住的是，您不能拥有 __try/__except 处理程序无处不在（C++ 展开和 C++ 异常处理程序与 SEH 不兼容），因此您必须编写一个中间函数来捕获 SEH 异常。

您的崩溃与线程相关吗？

最后但并非最不重要的一点是，“我只发生了 5% 的时间”症状可能是由不同的代码路径执行引起的，或者是多个线程一起处理相同的数据。

InterlockedDecrement 部分让我烦恼：是你的对象生活在多个线程中吗？ m_nRefCount 是否正确对齐且易失性 LONG ？

正确对齐和 LONG 部分在这里很重要。

如果您的变量不是 LONG（例如，它可能是 size_t，但在 64 位 Windows 上不是 LONG） ，那么该函数很可能会以错误的方式工作。

对于未在 32 字节边界上对齐的变量也是如此。你的代码中有#pragma pack()指令吗？您的 projet 文件是否更改了默认对齐方式（我假设您正在使用 Visual Studio）？

对于易失性部分，InterlockedDecrement似乎会生成读/写内存屏障，因此易失性部分不应该是强制性的（请参阅http://msdn.microsoft.com/en-us/library/f20w0x5e.aspx）。