Question

我知道已经有至少十几个valgrind reports invalid read个问题，但请耐心等待我，因为我真的不知道如何帮助自己，所以我要求你的。

我正在为OpenCV特征检测和特征描述模块编写一个包装器（我希望能够在某些时候实现我自己的专长检测/提取）。因此，我无法直接在OpenCV数据类型上运行。

因此，在从要素中提取描述符时，我将其存储为std::vector <std::vector <double> >而不是cv::Mat。我有这段代码，我首先计算描述符，然后将它们从一种符号转换为另一种符号：

// private
void CVDescriptor::calculateDescriptors(std::vector <cv::KeyPoint> &feats){
    this->feats = &feats;
    this->descCalc->compute(*(this->image), feats, this->desc);
    this->calculated = true;  
}

// public
void CVDescriptor::calculateDescriptors
                         (std::vector< std::vector< double > >& desc,
                          std::vector< cv::KeyPoint >& feats){    

    if (!this->calculated)
        this->calculateDescriptors(feats);

    assert(this->calculated);

    const double *temp;
    desc.clear();
    desc.reserve(this->desc.rows);
    for (int i=0, szi = this->desc.rows; i < szi; ++i){
        temp = this->desc.ptr<double>(i);

        // this line is the problem
        desc.push_back(std::vector<double>(temp, temp+(this->desc.cols)));
        //  .
        // /|\
        //  |
    }

    assert(desc.size() == this->desc.rows);
    assert(desc[0].size() == this->desc.cols);
    return;
}

以下是我的成员变量的类型，我已经检查并编写了初始化它们的地方（只是为了避免混淆）：

std::vector <cv::KeyPoint> *feats
cv::Mat *image;
    // it is set just before calling calculateDescriptors(desc, feats)
cv::Mat desc;
bool calculated; // set in the only constructor

这是OpenCV documentation for cv::DescriptorExtractor::compute。从我所看到的，每个计算的描述符应该是cv::Mat中的一行，并且应该具有与矩阵具有列一样多的组件。

我怀疑代码中某些地方的内存泄漏，所以我通过 Valgrind 运行它。它报告的第一件事是Invalid read of size 1在我的代码摘录中标有大箭头的行上。据我所知，每次调用CVDescriptor::calculateDescriptors(..)只报告两次，而不是for loop的每次迭代。

任何人都可以看到我的复制代码有什么明显错误吗？或者有任何其他想法如何发生这种情况？

如果需要，我可以提供其他信息，但我已经尝试将所有相关代码放在这里（因为我的项目很大）。提前谢谢大家（我很抱歉这个冗长的问题）......

Answer 1

我添加了每次迭代中复制的内存块的起始地址和结束地址的打印输出，这表明存在问题。摘自打印输出：

copied from 0xc0d5990 -- 0xc0d5d90
copied from 0xc0d5b90 -- 0xc0d5f90
copied from 0xc0d5d90 -- 0xc0d6190
copied from 0xc0d5f90 -- 0xc0d6390
copied from 0xc0d6190 -- 0xc0d6590

在每次迭代中，我都不小心尝试一次复制cv::Mat的两行，因为我通过double指针访问它，而存储的数据是float。

将temp声明为const float *temp;并更改<{p>}中的temp作业

temp = this->desc.ptr<float>(i);

诀窍。

OpenCV提取的描述符会导致valgrind在复制时报告无效读取

1 个答案: