游泳鱼0.66版本关于链表释放的疑问

蓝烟清

这几天一直很郁闷，游戏老跳出，无任何提示。找了很久，原来是链表释放内存的问题，很不解，先给源码（在JY_PicInit函数）
清除表面内容

1. //如果链表不为空，删除全部链表
   
2.     list_for_each_safe(pos,p,&cache_head){
   
3.         struct CacheNode *tmp= list_entry(pos, struct CacheNode , list);
   
4.                 int picid = tmp->id;
   
5.                 int fileid = tmp->fileid;
   
6.         if(tmp->s!=NULL)
   
7.                         SDL_FreeSurface(tmp->s);       //删除表面
   
8.                 list_del(pos);
   
9.                 SafeFree(tmp);
   
10. 
    
11.         }

复制代码

清除pic内存

1. for(i=0;i<PIC_FILE_NUM;i++){
   
2.         pic_file[i].num =0;
   
3.         SafeFree(pic_file[i].idx);
   
4.         SafeFree(pic_file[i].grp);
   
5.         SafeFree(pic_file[i].pcache);
   
6.         if(pic_file[i].fp){
   
7.             fclose(pic_file[i].fp);
   
8.             pic_file[i].fp=NULL;
   
9.         }
   
10. 
    
11.     }

复制代码

很关键的一点：在清除PIC内存的时候，没有循环pcache进行free

或许有人认为在清除表面的时候已经SafeFree(tmp);，但是实际上并没有
可以把内容再打印出来

1. //如果链表不为空，删除全部链表
   
2.     list_for_each_safe(pos,p,&cache_head){
   
3.         struct CacheNode *tmp= list_entry(pos, struct CacheNode , list);
   
4.                 int picid = tmp->id;
   
5.                 int fileid = tmp->fileid;
   
6.         if(tmp->s!=NULL)
   
7.                         SDL_FreeSurface(tmp->s);       //删除表面
   
8.                 list_del(pos);
   
9.                 SafeFree(tmp);
   
10. 
    
11.                 if(pic_file[fileid].pcache && pic_file[fileid].pcache[picid]){
    
12.                         JY_Debug("list  free cache %d fileid=%d  failed", picid, fileid);
    
13.                 }
    
14.         }

复制代码

在debug.txt就会看到这样类似的东西
11:46:45 list  free cache 36 fileid=0  failed
11:46:45 list  free cache 76 fileid=0  failed
11:46:45 list  free cache 1157 fileid=0  failed
11:46:45 list  free cache 1622 fileid=0  failed
11:46:45 list  free cache 1181 fileid=0  failed
11:46:45 list  free cache 1151 fileid=0  failed
11:46:45 list  free cache 1143 fileid=0  failed


说明实际上在清除链表的表面内容时，并没有把pache同时删除。

我是这样理解的
这样就会造成在切换场景（JY_PicInit函数在每切换一次场景时就调用）的时候，不断的造成内存泄漏，而且经过测试，cache下挂的Surface也没有被清除

不知道理解对不对


还有一点
if(tmp->s!=NULL)
                        SDL_FreeSurface(tmp->s);       //删除表面

应该也要改成
if(tmp->s!=NULL) {
                        SDL_FreeSurface(tmp->s);       //删除表面
                   tmp->s = NULL;  //避免野指针
}



List的作用是为了防止Surface过多，造成内存消耗过大。 但是遍历清空时这个为什么清不掉，原因为何，搞不明白啊

假修改器

你如果不介意，现在可以打包发过来吗？ 可以帮你查一查 qq 离线文件 78265172，借以就算了哈哈

鲜橙多

SafeFree(tmp) 等于
if (tmp)
{
   free(tmp);
   tmp = NULL;
}
所以这里不会有错，不会产生野指针。你贴出这段代码我看不出有问题。
你好好检测你之前传的tmp是否是一个有效的表面，如果是无效表面，c里面这样释放的话就会有可能造成程序崩溃。