c++ - 循环中的 std::unique_ptr - 内存泄漏

Question

下面是否按预期正确使用了 unique_ptr？代码以一些内存泄漏结束(可能是误报？还是真正的泄漏？)。我想将所有权移至 RunSimulation，其中 unique_ptr 的生命将结束，并且将在循环中创建更新的，然而，这以访问冲突结束。

理想情况下，我希望 main() 中的一个 unique_ptr 对象在 main() 内过期，而另一个发送到 main 外部以在 RunSimulation() 中释放。

class Result { public: int n; };

void RunSimulation(std::unique_ptr<Result> result) {result->n = 0;}

void main()
{
        boost::thread_group threads;

        std::unique_ptr<Result> r;
        std::unique_ptr<Simulation> sim = std::make_unique<Simulation>();

        for (int i = 0; i < 10; i++)
        {
            r = std::unique_ptr<Result>(new Result);

            //Erroneous lines:
            //threads.create_thread(boost::bind(&RunSimulation, std::move(r)));
            //threads.create_thread([&] {RunSimulation(std::move(r));  });
        }

        threads.join_all();
}

Answer 1

您没有内存泄漏，但这并不是说您的代码是正确的。看看你的循环:

for (int i = 0; i < 10; i++)
{
    r = std::unique_ptr<Result>(new Result);
    threads.create_thread([&] { RunSimulation(r.get()); });
}

每次您将一个新的结果实例分配给 r 时，旧的实例就会被 unique_ptr 删除。这可能发生在获取其指针的线程使用它之前，在这种情况下，您正试图取消引用已删除的内存。这是未定义的行为。

更新这是一个例子

#include <thread>
#include <memory>
#include <iostream>

class Result
{
public:
    Result() : x(0) { }
    int x;
};


void RunSimulation(std::unique_ptr<Result> result)
{
    result->x = 10;
    std::cout << result->x << std::endl;
}


int main()
{
    std::unique_ptr<Result> result = std::make_unique<Result>();
    std::thread t(RunSimulation, std::move(result));
    t.join();
}