线程创建子线程，线程退出时，释放了内存，但是TOP中只看到有一般的内存被释放了

korpus

架构是这样的。

主进程先初始化了一个线程池A，当其中一个线程接收到任务的时候，就会去创建几个子线程B来处理任务。（该A1线程使用pthread_jion等待子线程B退出，子线程用pthread_exit退出）。在子线程B中，使用了malloc函数连续分配了2段内存。任务处理完后，在退出pthread_exit前，释放他们。

其中的异常现象是这样的，如果我在子线程B中加了日志打印的函数，那么所有子线程退出时，上面申请的两个连续内存有一半的大小在top中显示为未释放（实际上，已经释放了，我向那2段内存中memcpy程序core了）。如果我在子线程B中不加日志打印函数，就不会有这个现象。

日志打印函数，检查过，没有什么问题。

1. 
   
2. #define LOG_INFO        Log_SysInfo(LOG_LEVEL_INFO,__FILE__,__LINE__),Log_Msg
   
3. 
   
4. void Log_SysInfo(const int iLogLevel, const char *pszFileName, const int iNum)
   
5. {
   
6.     int i = 0;
   
7.     char szLogTime[30];
   
8. 
   
9.     //  加一个写日志的线程锁
   
10.     pthread_mutex_lock(&g_WriteLog);
    
11. 
    
12.     memset(szLogTime, 0, sizeof(szLogTime));
    
13.     memset(g_szLogHead, 0, sizeof(g_szLogHead));
    
14. 
    
15.     g_iLogLevel = iLogLevel;
    
16. 
    
17.     //  如果设置的日志级别小直接返回
    
18.     if(g_iSetLogLevel < g_iLogLevel)
    
19.     {
    
20.         return;
    
21.     }
    
22. 
    
23.     Log_Check();
    
24. 
    
25.     timev_GetLogTime(szLogTime);
    
26.     snprintf(g_szLogHead, sizeof(g_szLogHead), "%s %s L=%d T=%ld E=%d ", szLogTime, pszFileName,
    
27.                  iNum, pthread_self(), errno);
    
28. 
    
29.     switch(iLogLevel)
    
30.     {
    
31.         case LOG_LEVEL_ERROR:
    
32.              sprintf(g_szLogHead + strlen(g_szLogHead), "%s", "[ERROR] ");
    
33.              break;
    
34.         case LOG_LEVEL_WARN:
    
35.              sprintf(g_szLogHead + strlen(g_szLogHead), "%s", "[WARN] ");
    
36.              break;
    
37.         case LOG_LEVEL_DEBUG:
    
38.              sprintf(g_szLogHead + strlen(g_szLogHead), "%s", "[DEBUG] ");
    
39.              break;
    
40.         case LOG_LEVEL_INFO:
    
41.              sprintf(g_szLogHead + strlen(g_szLogHead), "%s", "[INFO] ");
    
42.              break;
    
43.         case LOG_LEVEL_ALL:
    
44.              sprintf(g_szLogHead + strlen(g_szLogHead), "%s", "[ALL] ");
    
45.              break;
    
46.         default:
    
47.              break;
    
48.     }
    
49. }
    
50. 
    
51. void Log_Msg(const char *format, ...)
    
52. {
    
53.     FILE *fp;
    
54.     va_list args;
    
55.     int  olderrno;
    
56.     //char szContext[1024*10];    //  分配10K的临时变量空间,如果用内存分配会不会影响性能
    
57.     int i;
    
58. 
    
59.     //  如果设置的日志级别小,则直接返回
    
60.     if(g_iSetLogLevel < g_iLogLevel)
    
61.     {
    
62.         pthread_mutex_unlock(&g_WriteLog);
    
63.         return;
    
64.     }
    
65.     olderrno = errno;     
    
66.     //memset(szContext, 0, sizeof(szContext));
    
67. 
    
68.     if(NULL != (fp = fopen(g_szLogFile, "a+")))
    
69.     {
    
70.         fprintf(fp, "%s", g_szLogHead);
    
71.         va_start(args, format);
    
72. //        fprintf(fp, "%s\n", szContext);
    
73.         vfprintf(fp, format, args);
    
74. //        fflush(fp);
    
75.         va_end(args);
    
76.         fprintf(fp, "%s", "\n");
    
77.         fclose(fp);
    
78.     }
    
79.     else
    
80.     {
    
81.         printf("fopen error, errno=%d, file=%s\n", errno, g_szLogFile);
    
82.         perror("fopen");
    
83.         //printf("context1 [%s], file[%s:%d] \n", szContext, __FILE__, __LINE__);
    
84.     }
    
85.     errno = olderrno;
    
86. 
    
87.     pthread_mutex_unlock(&g_WriteLog);
    
88. }

复制代码

korpus

这个是子线程的代码，就算是这样，先连续申请两段内存，然后释放，top中显示就没有释放完。  如果我在pReadMemory申请后，再释放，pWriteMemory申请后，再释放就是正常的。搞不懂是什么意思，求大哥大姐帮帮忙！

1. void *trans_SingleThread(void *pData)
   
2. {
   
3.     int iRet;
   
4.     MultTransData *pMultTransData = (MultTransData *)pData;
   
5.     MMapData *pMMapData = NULL;
   
6.     ListNode *pTemp = NULL;
   
7.     void *pReadMemory = NULL;
   
8.     void *pWriteMemory = NULL;
   
9.     MMapData *pFirstSection = NULL;
   
10. 
    
11.     pTemp = pMultTransData->m_pMMapDataList;
    
12.     pFirstSection = (MMapData *)pTemp->pData;
    
13. 
    
14. 
    
15.     //  为写空间分配内存地址
    
16.     pWriteMemory = ufs_MallocMemory(pFirstSection->m_lWriteTotalSize);
    
17.     if(NULL == pWriteMemory)
    
18.     {
    
19.         LOG_ERROR("Malloc Memory Failure!");
    
20.         ufs_vFreeNormalMemory(&pReadMemory);
    
21.         iRet = RT_FAILURE;
    
22.         pthread_exit((void *)&iRet);
    
23.     }
    
24. 
    
25.     //  为读空间分配内存地址
    
26.     pReadMemory = ufs_MallocMemory(pFirstSection->m_lReadTotalSize);
    
27.     if(NULL == pReadMemory)
    
28.     {
    
29.         LOG_ERROR("Malloc Memory Failure!");
    
30.         iRet = RT_FAILURE;
    
31.         pthread_exit((void *)&iRet);
    
32.     }
    
33. 
    
34. 
    
35. LOG_DEBUG("Thread Will Quit");
    
36. 
    
37. ufs_vFreeNormalMemory(&pReadMemory);
    
38. ufs_vFreeNormalMemory(&pWriteMemory);
    
39. 
    
40. 
    
41. 
    
42. pthread_exit((void *)&iRet);
    
43. 
    
44. }

复制代码

korpus

求帮助！有没有哪个可以帮忙看看呢

gaojl0728

回复 3# korpus


    1. 能不能把LOG_DEBUG的代码贴出来。
2. 你是看top的哪一个字段？
3. 你说“我向那2段内存中memcpy程序core了”， 什么叫core了， 是crash了吗？
4. crash了， 不一定能证明就是释放了， 把memcpy的代码贴出来

korpus

回复 4# gaojl0728


    你好，谢谢你的关注，
1 LOG_DEBUG，贴出来了 ，就在第一个代码里面的
2 top的res字段
3 意思好像没有表达清楚，意思是，在我free那2段内存后，为了检查是否真的释放了，然后向释放的内存memcpy内容，然后程序crash了
4 memcpy(pReadMemory, "12345", 5);

gaojl0728

1. 你第一行贴的是LOG_INFO， 我想看看LOG_DEBUG。
2. ufs_vFreeNormalMemory传递的为什么是双重指针，难道会把参数设置为NULL?
如果是写NULL的话，你就不需要memcpy测试了， 直接看看是不是NULL就行了
3. 能不能把ufs_MallocMemory和ufs_vFreeNormalMemory的代码贴出来看看？
如果实现是用的malloc和free, 那内存可能被glibc的内存池回收了，并没有还给内核。把代码贴出来看看吧
4. pFirstSection->m_lWriteTotalSize和pFirstSection->m_lReadTotalSize的大小多少？

gaojl0728

回复 5# korpus


    1. 你第一行贴的是LOG_INFO， 我想看看LOG_DEBUG。
2. ufs_vFreeNormalMemory传递的为什么是双重指针，难道会把参数设置为NULL?
如果是写NULL的话，你就不需要memcpy测试了， 直接看看是不是NULL就行了
3. 能不能把ufs_MallocMemory和ufs_vFreeNormalMemory的代码贴出来看看？
如果实现是用的malloc和free, 那内存可能被glibc的内存池回收了，并没有还给内核。把代码贴出来看看吧
4. pFirstSection->m_lWriteTotalSize和pFirstSection->m_lReadTotalSize的大小多少？

korpus

回复 7# gaojl0728


    你好，首先谢谢您哈。
1. 你第一行贴的是LOG_INFO， 我想看看LOG_DEBUG。

1. #define LOG_ERROR        Log_SysInfo(LOG_LEVEL_ERROR,__FILE__,__LINE__),Log_Msg
   
2. #define LOG_WARN        Log_SysInfo(LOG_LEVEL_WARN,__FILE__,__LINE__),Log_Msg
   
3. #define LOG_INFO        Log_SysInfo(LOG_LEVEL_INFO,__FILE__,__LINE__),Log_Msg
   
4. #define LOG_DEBUG        Log_SysInfo(LOG_LEVEL_DEBUG,__FILE__,__LINE__),Log_Msg

复制代码

2. ufs_vFreeNormalMemory传递的为什么是双重指针，难道会把参数设置为NULL?
如果是写NULL的话，你就不需要memcpy测试了， 直接看看是不是NULL就行了

是的，就是为了将其置为空，打印过，释放后，确实也是NULL

3.函数如下

1. void *ufs_MallocMemory(const long lSize)
   
2. {
   
3.     void *pAddr = NULL;
   
4.     if(lSize <= 0)
   
5.     {
   
6.         LOG_ERROR("malloc memory size less than zero");
   
7.         return NULL;
   
8.     }
   
9. 
   
10.     pAddr = malloc(lSize);
    
11.     if(NULL == pAddr)
    
12.     {
    
13.         LOG_ERROR("malloc memory size error!, malloc size(%d)", lSize);
    
14.         return NULL;
    
15.     }
    
16. 
    
17.     memset(pAddr, 0, lSize);
    
18. 
    
19.     return pAddr;
    
20. }
    
21. 
    
22. void ufs_vFreeNormalMemory(void **pAddr)
    
23. {
    
24. //    LOG_WARN("TEST MEMORY FREE[%p]", *pAddr);
    
25. 
    
26.     if(NULL == *pAddr)
    
27.         return;
    
28.     else
    
29.         free(*pAddr);
    
30.     *pAddr = NULL;
    
31. }
    

复制代码

是用的malloc与free，在网上也查了说有可能被glibc回收，但是每次我运行一次的时候，内存就增加了。

4. pFirstSection->m_lWriteTotalSize和pFirstSection->m_lReadTotalSize的大小多少？
这几个值是设定的值，一般在10M以上

再次谢谢哈

folklore

内存的问题我不猜～～



>>但是每次我运行一次的时候，内存就增加了。

如果是指进程已经结束重跑的话。
就不用担心内存问题了。
进程一结束，
渗漏的内存也会被OS自动回收。

gaojl0728

回复 8# korpus

glibc只是一个可疑的点，但是不确定是不是真的被内存池回收而没换给内核了。
看你的代码没什么问题， 没有内存泄露， 也没看到死锁
如果不是被内存池缓存的话，我感觉可能只是一个top的显示问题