libdecodeqr_0.9.3.orig 内存泄漏修改

小咪咪 2022-07-14 03:45 15阅读 0赞

### libdecodeqr\_0.9.3.orig ###

[libdecodeqr下载介绍][libdecodeqr]  
这个开源库是一个日本人写的，可以识别QR 二维码识别的。  
还有两个二维码识别的库，zbar和zxing。自行寻找

这篇文章主要说说libdecodeqr\_0.9.3.orig 库存在内存泄漏的问题。  
花了几天逐句排除，总算皇天不负苦心人。

1 作者喜欢把变量内存的释放放在析构函数里面，这样设计代码，的确很好，但是导致局部测试时好麻烦，要去思考每个函数调用完，存在哪些内存申请，哪些需要释放。

2 导致原代码库有内存泄漏主要为以下几点：

1.  作者定义了两个内存管理器作为成员变量，CvMemStorage *\_stor， CvMemStorage \*\_stor\_tmp。然后代码里面，局部变量例如hull使用完这个内存管理器后，用cvRelease((void \**)&hull)来完成hull内存释放，而\_stor和\_stor\_tmp则一直被各种局部变量反复调用。每次调用完都用用cvRelease来释放对应的局部变量。\_stor和\_stor\_tmp则在析构函数中完成最后的释放。
2.  上面是我根据作者的代码大致能想到的解释，说了不要打我，但是问题来，如果用一个while套住如下执行语句，发现内存会泄漏。因为我认为 cvRelease((void *\*)&markers\_vertex)这个释放内存的方法应该不适合CvSeq* 。网上搜索会知道其实stor\_tmp\_markers\_vertex和markers\_vertex公用一块内存。可以用cvReleaseMemStorage(&stor\_tmp\_markers\_vertex);来释放，此时这两个变量就都失去了内存。当然也可以用cvReleaseMemStorage(&markers\_vertex->storage);

CvMemStorage *stor_tmp_markers_vertex = NULL;
                stor_tmp_markers_vertex = cvCreateMemStorage(0);
    
                while(1){
                CvSeq *markers_vertex = cvCreateSeq(CV_SEQ_ELTYPE_POINT, sizeof(CvSeq),
                    sizeof(CvPoint),
                    stor_tmp_markers_vertex);
                cvRelease((void **)&markers_vertex)
                }

1.  因此我的第一个改动是：
    
    this->\_find\_code\_area\_contour(FIND\_CODE\_AREA\_POLY\_APPROX\_TH);函数发现有问题
    
    ：破釜沉舟删除CvMemStorage \*\_stor;  
    CvMemStorage \*\_stor\_tmp;然后源码中用到这两个变量，全部单独配一个内存管理器，使用完就删除。PS：因为\_seq\_code\_area\_contour要多处使用，因此给他单独创建对应的成员变量CvMemStorage。

2 第二个改动地方：  
this->\_get\_format\_info(code\_matrix, 0);函数发现有问题

发现里面的成员函数对应的析构函数不规范。

1.  
Field::~Field()  
\{

for (int i = 0; i < this->_pool_size; i++)
        {
            if (this->pool[i]){
                delete this->pool[i];
            }
        }
        delete [] this->pool;
        if(this->_need_delete){
            delete this->exp2vect;
            delete this->vect2exp;
        }
    }

1.  在代码 if(hull)\{\} 里面有申请 CvMat\* vector=cvCreateMat(1,hullcount,CV\_32SC2);因此需要在最后添加 cvReleaseMat(&vector);释放内存
2.  [修改后的源码下载][Link 1]

[libdecodeqr]: http://trac.koka-in.org/libdecodeqr
[Link 1]: http://download.csdn.net/detail/zhang2012liang/9689024