转自:http://blog.sina.com.cn/s/blog_4298002e01019gle.html
很早以前就在使用wuchangchang开发的SiftGPU库,以前就经常出现运行完Sift后程序崩溃的问题,但是一直没有空闲去跟踪这个问题的所在。直到昨天才修正了这个bug。
仔细阅读SiftGPU的代码可以发现,SiftGPU的CreateContextGL()函数调用GlobalUtil类的静态方法GlobalUtil::CreateWindowEZ()创建了一个静态的LiteWindow对象实例,供GLSL程序使用。代码在GlobalUtil.cpp内,如下:
int GlobalUtil::CreateWindowEZ()
{
static LiteWindow window;
return CreateWindowEZ(&window);
}
注意,这个LiteWindow对象不是以指针形式声明的,而且整个程序再没有销毁过该LiteWindow对象,即便SiftGPU被销毁之后,这个对象仍然存在,直到包含SiftGPU的对象被析构或者包含SiftGPU的函数退出——此时LiteWindow对象依然存在,但是其GL上下文已经丢失,相关指针变为野指针。
这种机制为后面的程序带来了隐藏的bug:如果我们在主程序结束前都不析构包含SiftGPU的对象,等到主程序结束时再由系统自动销毁,则程序不会出现任何问题。但是如果我们一运行完SiftGPU的功能就把包含SiftGPU的对象析构了,然后再等主程序结束,就会出现LiteWindow对象无法被析构的错误——因为该对象已经在析构包含SiftGPU的对象时产生了野指针,系统再对其回收就会出现问题。
其实wuchangchang的这个代码本意是把GlobalUtil类设计成一个单体(singleton),只创建一个LiteWindow实例。然而由于该对象不是以指针形式声明的,所以导致无法主动销毁。为了修正这个bug,我们需要对GlobalUtil的代码进行一些修改,将其变成一个完全的单体模式。修改方法如下:
1)打开GlobalUtil.h,删除static int CreateWindowEZ(LiteWindow* window)方法,并把static int CreateWindowEZ()方法改为:
static LiteWindow* CreateWindowEZ();
添加一个销毁实例的方法:
static void DestroyWindowEZ();
添加一个保护的静态成员变量:
protected:
static LiteWindow *m_pWindow;
2)打开GlobalUtil.cpp,删除int GlobalUtil::CreateWindowEZ(LiteWindow* window)和int GlobalUtil::CreateWindowEZ()的代码,添加如下代码:
LiteWindow *GlobalUtil::m_pWindow = NULL;
LiteWindow* GlobalUtil::CreateWindowEZ()
{
if (m_pWindow == NULL)
{
m_pWindow = new LiteWindow;
}
if (!m_pWindow->IsValid())
{
m_pWindow->Create(_WindowInitX, _WindowInitY, _WindowDisplay);
}
if(m_pWindow->IsValid())
{
m_pWindow->MakeCurrent();
}
else
{
std::cerr << "Unable to create OpenGL Context!\n";
std::cerr << "For nVidia cards, you can try change to CUDA mode in this case\n";
}
return m_pWindow;
}
void GlobalUtil::DestroyWindowEZ()
{
if (m_pWindow != NULL)
{
delete m_pWindow;
m_pWindow = NULL;
}
}
3)在SiftGPU.h中的SiftGPU类和SiftMatchGPU类分别添加:
SIFTGPU_EXPORT virtual void DestroyContextGL();
4)在SiftGPU.cpp中添加:
void SiftGPU::DestroyContextGL()
{
return GlobalUtil::DestroyWindowEZ();
}
在SiftMatchGPU.cpp中添加:
void SiftMatchGPU::DestroyContextGL()
{
GlobalUtil::DestroyWindowEZ();
}
5)修改SiftGPU.h中的SIFTGPU_EXPORT int CreateLiteWindow(LiteWindow* window)为:
SIFTGPU_EXPORT LiteWindow* CreateLiteWindow();
6)修改GlobalUtil.cpp中的int CreateLiteWindow(LiteWindow* window)为:
LiteWindow* CreateLiteWindow()
{
return GlobalUtil::CreateWindowEZ();
}
7)在自己的代码中使用完Sift的地方,手动调用析构GL context的函数,如下:
假设变量:SiftGPU *pSift;
pSift->DestroyContextGL();
或者:SiftMatchGPU *pSiftMatch;
pSiftMatch->DestroyContextGL();
具体调用哪句,视最后使用哪个对象而定。
自此,我们的程序就不会出现重复释放LietWindow的错误了。