c++ - 在单个 slot_type 上使用多个 boost::connect 的内存泄漏

Question

当我尝试将多个信号连接到单个 slot_type 时，我正在使用 boost::signals 并泄漏内存。我在各种论坛上看到过同样的泄漏报告，但找不到任何提到正确的方法或任何解决方法的信息。

我正在尝试做的事情:

我试图将 boost::bind() 的结果传递给一个函数。在此函数中，我想将多个信号连接到该结果。第一个连接工作正常，但第一个连接之后的每个连接都会泄漏一个句柄。

下面是一些示例代码:

typedef boost::signal0<void> LeakSignalType;

class CalledClass
{
   /* ... */
   void connectToSlots(LeakSignalType::slot_type &aSlot)
   {
        LeakSignalType *sig;
        std::list<LeakSignalType*> sigList;
        std::list<LeakSignalType*>::iterator sigIter;

        for(int i = 0; i < 50; i++)
        {
            /*Connect signals to the passed slot */
            sig = new LeakSignalType;
            sig->connect(aSlot);
            sigList.push_back(sig);
        }
        for(sigIter = sigList.begin(); sigIter != sigList.end(); sigIter++)
        {
            /* Undo everything we just did */
            delete *sigIter;
        }
        sigList.clear();
        /*Everything should be cleaned up now */
   }
   /* ... */
}

class CallingClass : public boost::signals::trackable
{
   CalledClass calledInstance;
   /* ... */
   void boundFunction(int i)
   {
   /*Do Something*/
   }

   void connectSignals()
   {
       calledInstance.connectToSlots(boost::bind( &CallingClass::boundFunction, this, 1));
   }
   /* ... */
};

现在调用 CallingClass::connectSignals()。

我希望对 connectToSlots 的调用会将 50 个信号连接到一个插槽，然后断开连接并清除所有这些信号。实际发生的是 1 个信号完全清理，然后其余 49 个部分清理，但泄漏了一些内存。

将插槽传递给函数以多次使用的正确方法是什么？任何帮助将不胜感激。

谢谢，克里斯

Answer 1

我很确定这是一个错误。如果你把它分解成一个小例子，例如:

void boundFunction(int) { }
typedef boost::signal0<void> LeakSignalType;
LeakSignalType::slot_type aSlot = boost::bind(&::boundFunction, 1);
LeakSignalType sig1, sig2;
sig1.connect(aSlot);
sig2.connect(aSlot);

并跟踪分配情况，您会发现在 boost/lib/的第 75 行分配了一个对象(boost::signals::detail::signal_base_impl::iterator) signals/src/signal_base.cpp 没有被释放。

// Allocate storage for an iterator that will hold the point of
// insertion of the slot into the list. This is used to later remove
// the slot when it is disconnected.
std::auto_ptr<iterator> saved_iter(new iterator);

在第一个 connect 上，此迭代器附加到一个新的连接对象，其中 signal_data 为 NULL:

data->watch_bound_objects.get_connection()->signal_data =
  saved_iter.release();

然而，在第二个connect 上，同一个连接对象被重用，同一行盲目地覆盖了原来的指针值。第二个对象被清理，但第一个没有。

作为验证，signal_base_impl::slot_disconnected 中的断点，唯一清除 signal_data 的地方，只触发一次。

我在 1.39.0 中追踪到这个，但它看起来在 1.40.0 中是一样的。

您可以修改 boost::signals::detail::signal_base_impl::connect_slot 以清除它在现有连接的 signal_data 字段中找到的任何先前迭代器值，如果您愿意进行此类更改并运行自定义构建的 Boost。

最好只确保您只设置这些设置次数固定，并忍受一些您知道不会随着时间增长的小内存泄漏。

更新:

我打算将其提交给 Boost 错误跟踪器，但它已经存在了。然而，这是一个小得多的测试用例。

https://svn.boost.org/trac/boost/ticket/738

3 年前开业，未分配到任何里程碑:-[