SKIA源代码发现的两处内存泄露及解决方法!

[warpo]

最近在移植第三方SKIA图形库.发现有内存泄露.简述如下.

SKIA源代码有内存泄露的解决办法.

1) 源码SkMemory_stdlib.cpp中将函数 static SkMutex& get_block_mutex() 修改如下:

static SkMutex& get_block_mutex() {

//fixed by warpo. ADD 35-40.
static SkMutex gbm;
static SkMutex* gBlockMutex = &gbm;
if (NULL == gBlockMutex ) {
gBlockMutex = new SkMutex;
}
return *gBlockMutex;

//原始代码如下:
//static SkMutex* gBlockMutex;
//if (NULL == gBlockMutex) {
// gBlockMutex = new SkMutex;
//}
//return *gBlockMutex;
}

该错是new SkMutex之后未释放内存.



2).源码SkGlyphCache.cpp中的内存泄露.

为SkGlyphCache类增加一个静态函数,然后在程序结束时调用即可:

size_t SkGlyphCache::freeGlyphCache_Globals(/*SkGlyphCache_Globals* globals, size_t bytesNeeded*/)
{

SkGlyphCache_Globals& globals = FIND_GC_GLOBALS();
globals.validate();

size_t bytesFreed = 0;
int count = 0;

// don't do any "small" purges
//size_t minToPurge = globals.fTotalMemoryUsed >> 2;
//if (bytesNeeded < minToPurge)
// bytesNeeded = minToPurge;

SkGlyphCache* cache = FindTail(globals.fHead);
while (cache != NULL ) {
SkGlyphCache* prev = cache->fPrev;
bytesFreed += cache->fMemoryUsed;

#ifdef USE_CACHE_HASH
unsigned index = desc_to_hashindex(cache->fDesc);
if (cache == globals.fHash[index]) {
globals.fHash[index] = NULL;
}
#endif

cache->detach(&globals.fHead);
SkDELETE(cache);
cache = prev;
count += 1;
}

SkASSERT(bytesFreed <= globals.fTotalMemoryUsed);
globals.fTotalMemoryUsed -= bytesFreed;
globals.validate();

#ifdef SPEW_PURGE_STATUS
if (count) {
SkDebugf("purging %dK from font cache [%d entries]\n",
(int)(bytesFreed >> 10), count);
}
#endif

return bytesFreed;
}

在WIN平台下(其它平台未验证)引起该内存泄露的原因是SKIA使用的字体缓存机制,当绘制文字时都会new一些缓存,其中globals是全局静态的一个保存字体缓存数据的结构,但笔者使用源码时发现未释放.所以需要手动调用释放.

3). 需要手动调用类的SkTypefaceCache的purge方法清空缓存.该问题也是由于NEW后未释放造成的.具体流程不详说.可以自己跟踪调试.

[passself]

楼主厉害，修改framework的代码

[aSysBang]

不懂C
有个疑问，get_block_mutex() 被调用的地方都是
SkAutoMutexAcquire ac(get_block_mutex()); //不知道ac 这个函数啥意思。。。

而SkAutoMutexAcquire的析构函数中有
~SkAutoMutexAcquire()
{
if (fMutex)
fMutex->release();
}

那么SkAutoMutexAcquire在释放时 是不是也会释放这个指针

[warpo]

[Quote=引用 2 楼 l417584711 的回复:]
不懂C
有个疑问，get_block_mutex() 被调用的地方都是
SkAutoMutexAcquire ac(get_block_mutex()); //不知道ac 这个函数啥意思。。。

而SkAutoMutexAcquire的析构函数中有
~SkAutoMutexAcquire()
{
if (fMutex)
fMutex->release();
}……
[/Quote]
之前我和你一样,注意到它的析构,不过源码里的fMutex->release()并未delete fMutex.你可以去看看内容,它只是做了线程相关的互斥量的解锁.并未释放内存.
而源码中
static SkMutex* gBlockMutex;
if (NULL == gBlockMutex) {
gBlockMutex = new SkMutex;
}
return *gBlockMutex
静态局部变量至程序结束时都会保存第一次new SkMutex的指针变量,但结束程序并未释放这个指针变量所指对象.
get_block_mutex() 被调用的地方就是SkAutoMutexAcquire ac(get_block_mutex());
ac并不是函数,是SkAutoMutexAcquire 类对象.括号内的是SkAutoMutexAcquire类对象ac的构造函数的参数.这个参数是get_block_mutex(),返回的就是new SkMutex.

[csuhanyong]

楼主问下，
skia在decode一张图片时，如何让其尽量少的占用内存，图片显示的质量差点没关系。

[qthsrs232]

只会修修改改