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

Question

最近在移植第三方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后未释放造成的.具体流程不详说.可以自己跟踪调试.

Answer 1

楼主厉害，修改framework的代码

Answer 2

不懂C
有个疑问，get_block_mutex()  被调用的地方都是
SkAutoMutexAcquire  ac(get_block_mutex());   //不知道ac 这个函数啥意思而SkAutoMutexAcquire的析构函数中有
    ~SkAutoMutexAcquire()
    {
        if (fMutex)
            fMutex->release();
    }那么SkAutoMutexAcquire在释放时 是不是也会释放这个指针

Answer 3

之前我和你一样,注意到它的析构,不过源码里的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.

Answer 4

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