如何消除 Qthreads 中的内存泄漏？ 12+Gb 泄漏

Question

我正在尝试加快我的 (qt c++ opencv) 程序，该程序应该计算照片中的颜色数量以供将来过滤。 单线程方式没有泄漏，但是速度很慢。

添加 8 个线程后，我已经将此过程加速了 5 倍。

当我将程序切换到多线程时，问题就开始了。

有一个巨大的内存泄漏！ http://snag.gy/cHRrS.jpg

遵循此建议 (https://stackoverflow.com/a/12859444) 我阻止了继承 QThread 并实现 run()。

这是一个 for 循环，用于计算每个移动 1 个像素的新图像中的每个像素：

ColorCounterController *cntrl[arrSize];

for (int i = 0; i < box; i++)//x
{
    for (int j = 0; j < box; ++j)//y
    {
        cv::Mat res=process(image,i,j);


        //Using 1 core
        //colors=ColorDetectController::getInstance()->colorsCount(res);

        //Using all 8 cores
        cntrl[cnt2%arrSize]= new ColorCounterController(res,this);

        ++cnt2;
    }

    ++cnt;
    emit setStatusProgressSignal((int)(cnt/amnt*100));
}

delete[] *cntrl;

评论：

当使用 1 个核心（以上代码）时，我有单例来运行 colorsCount(res) 函数。 在 8 个内核的情况下，我使用几乎相同的函数，但从 ColorCounterController 调用。

class ColorCounterController : public QObject{
Q_OBJECT
private:
QThread thread;
ColorCounter *colorCntr;
Pixalate *pixelate;
private slots:
void freecolorCntr(){
    delete colorCntr;
}
public:
ColorCounterController(const cv::Mat &image,Pixalate *pxobj) {
    colorCntr= new ColorCounter();
    colorCntr->setimageThread(image);
    colorCntr->moveToThread(&thread);
    connect(&thread, SIGNAL(started()), colorCntr, SLOT(colorsCountThread()));
    connect(colorCntr, SIGNAL(finished()), &thread, SLOT(quit()));
    connect(colorCntr, SIGNAL(finished()), colorCntr, SLOT(deleteLater()));
        connect(colorCntr, SIGNAL(results(int)), pxobj, SLOT(results(int)));
    thread.start();
}

    ~ColorCounterController() {
    thread.quit();
    thread.wait();
     qDebug() << QString("Controller quit wait");
    //delete colorCntr; //err
}

我想泄漏是在 ColorCounterController 构造函数中：

        colorCntr= new ColorCounter();

但是如何避免呢？此代码导致错误。 在析构函数中：

//delete colorCntr; //err

在构造函数中：

//connect(&thread, SIGNAL(finished()), &thread, SLOT(deleteLater()));

请帮忙！

附言

我改变了这个

delete[] *cntrl;

到这里

    for (int i = 0; i < arrSize; i++){
    if (cntrl[i])
        delete cntrl[i];
    }

cntrl[cnt2%arrSize]之前的所有指针都为NULL

没有改变

附言 如果您想为这个问题做出贡献： https://github.com/ivanesses/curiosity

Answer 1

2个问题导致泄漏：

1. 指向新 ColorCounterController 对象的指针将永远丢失（并且它们的内存泄漏），因为“for”循环的主体将运行 N 次 (N=box*box) 创建 N 个 ColorCounterController 对象但仅指向 8它们中的一部分将适合您稍后用于删除对象的数组。

2. cntrl 是一个指针数组。您需要遍历它并在它的每个元素上调用 delete（纯删除，而不是 delete[]）。

3. 由 OP 发现：必须使用 imageThread.release();而不是 imageThread.deallocate()