代码测试之内存泄露

  【 声明：版权所有，欢迎转载，请勿用于商业用途。  联系信箱：feixiaoxing @163.com】

    在我们个人编程的过程当中，内存泄露虽然不会像内存溢出那样造成各种莫名奇妙的问题，但是它的危害也是不可忽视的。一方面，内存的泄露导致我们的软件在运行过程中占用了越来越多的内存，占有资源而又得不到及时清理，这会导致我们程序的效率越来越低；另一方面，它会影响我们用户的体验，失去市场的竞争能力。

    常见的内存泄露是这样的：

view plain

1. void process(int size)  
2. {  
3.     char* pData = (char*)malloc(size);  
4.   
5.     /* other code  */  
6.       
7.     return; /* forget to free pData */  
8. }  

    如上图所示，我们在函数process的处理过程中，每一次都需要对内存进行申请，但是在函数结束的时候却没有进行释放。如果这样的一段代码出现在业务侧，那么后果是难以想象的。举个例子来说，如果我们服务器每秒钟需要接受100个用户的并发访问，每个用户过来的数据，我们都需要本地申请内存重新保存一份。处理结束之后，如果内存没有得到很好地释放，就会导致我们服务器可用的物理内存越来越少。一旦达到某一个临界点之后，操作系统不得不通过内外存的调度来满足我们申请新内存的需求，这在另一方面来讲又会降低服务器服务的质量。

    内存泄露的危害是不言而喻的，但是查找内存泄露却是一件苦难而且复杂的工作。我们都知道，解决bug是一件非常简单的事情，但是寻找bug的出处却是一件非常吃力的事情。因此，我们有必要在自己编写代码的时候，就把查找内存泄露的工作放在很重要的位置上面。那么有没有什么办法来解决这一问题呢？

    我想要做到解决内存泄露，必须做到下面两个方面：

    （1）必须记录内存在哪个函数申请的，具体文件的行数是多少

    （2）内存应该什么时候被释放

   要完成第1个条件其实并不困难。我们可以用节点的方法记录我们申请的内存：

    a）设置节点的数据结构

view plain

1. typedef struct _MEMORY_NODE  
2. {  
3.     char functionName[64];  
4.     int line;  
5.     void* pAddress;  
6.     struct _MEMORY_NODE* next;  
7.   
8. }MEMORY_NODE;  

    其中 functionName记录函数名称，line记录行数， pAddress记录分配的地址， next记录下一个内存节点。

    

    b）修改内存的分配函数

    对业务侧的malloc进行函数修改，添加下面一句宏语句

    #define malloc(param)  MemoryMalloc(__FUNCTION__, __LINE__, param)

    在桩函数侧书写下面的代码

view plain

1. void* MemoryMalloc(const char* name, int line, int size)  
2. {  
3.     void* pData = (void*)malloc(size);  
4.     MEMORY_NODE* pMemNode = NULL;  
5.     if(NULL == pData) return NULL;  
6.     memset((char*)pData, 0, size);  
7.   
8.     pMemNode = (MEMORY_NODE*)malloc(sizeof(MEMORY_NODE));  
9.     if(NULL == pMemNode){  
10.         free(pData);  
11.         return NULL;  
12.     }  
13.     memset((char*)pMemNode, 0, sizeof(MEMORY_NODE));  
14.     memmove(pMemNode->functionName, name, strlen(name));  
15.     pMemNode->line = line;  
16.     pMemNode->pAddress = pData;  
17.     pMemNode->next = NULL;  
18.     add_memory_node(pMemNode);  
19.   
20.     return pData;  
21. }  

    内存的分配过程中还涉及到了节点的添加，所以我们还需要添加下面的代码

view plain

1. static MEMORY_NODE* gMemNode = NULL;  
2.   
3. void add_memory_node(MEMORY_NODE* pMemNode)  
4. {  
5.     MEMORY_NODE* pNode = gMemNode;  
6.     if(NULL == pMemNode) return;  
7.     if(NULL == gMemNode){  
8.         gMemNode = pMemNode;  
9.         return;  
10.     }  
11.   
12.     while(NULL != pNode->next){  
13.         pNode = pNode->next;  
14.     }  
15.     pNode->next = pMemNode;  
16.     return;  
17. }  

    文中gMemNode表示所有内存节点的根节点，我们每增加一次malloc过程就会对内存节点进行记录。在记录过程中，我们还会记录调用malloc的函数名称和具体文件行数，这主要是为了方便我们在后面进行故障定位的时候更好地查找。

   完成了第一个条件之后，我们就要对第二个条件进行完成。

   a）内存什么时候释放，这取决于我们在函数中是怎么实现的，但是我们在编写测试用例的时候却是应该知道内存释放没有，比如说如果测试用例全部结束了，我们有理由相信assert(gMemNode == NULL)这应该是恒等于真的。

    b）内存释放的时候，我们应该做些什么？和节点的添加一样，我们在内存释放的时候需要free指定的内存，free节点，free节点的内存，下面就是在释放的时候我们需要进行的操作

    对业务侧的free函数进行修改，添加下面一句宏代码，

    #define free(param)      MemoryFree(param)

    在桩函数侧输入下面的代码：

view plain

1. void MemoryFree(void* pAddress)  
2. {  
3.     if(NULL == pAddress) return;  
4.     delete_memory_node(pAddress);  
5.     free(pAddress);  
6. }  

    在删除内存的时候，需要删除节点，删除节点的内存

view plain

1. void delete_memory_node(void* pAddress)  
2. {  
3.     MEMORY_NODE* pHead = gMemNode;  
4.     MEMORY_NODE* pMemNode = gMemNode;  
5.     while(NULL != pMemNode){  
6.         if(pAddress == pMemNode->pAddress)  
7.             break;  
8.     }  
9.     if(NULL == pMemNode) {  
10.         assert(1 == 0);  
11.         return;  
12.     }  
13.   
14.     while(pMemNode != pHead->next){  
15.         pHead = pHead->next;  
16.     }  
17.   
18.     if(pMemNode == gMemNode){  
19.         gMemNode = gMemNode->next;  
20.     }else{  
21.         pHead->next = pMemNode->next;  
22.     }  
23.     free(pMemNode);  
24.     return;  
25. }  

    有了上面一小段代码的帮助，我们在编写测试用例的时候，就可以在函数执行后，通过判断内存节点是否为空的方法判断内存是否已经释放。如果内存没有释放，我们还能通过节点的信息帮助我们是哪里发生了错误，但是这个方法还有两个缺点：

    （1）没有考虑缓存的情况，好多内存分配了之后并不会在函数中马上释放，而是放在缓存池中等待下一次调用，这就需要我们准确把握和判断了。

    （2）代码中节点删除和添加的时候没有考虑多进程的情形，应该考虑用一个互斥锁或者是信号量加以保护。

（全文完）