[已解决]线程的内存泄漏问题

2736946915

1. 
   
2. 
   
3. PostQueuedCompletionStatus(hIOCP, sizeof(IOCP_PARAM), (ULONG_PTR)new IOCP_PARAM(IocpListPop, "hellp world",func), NULL);
   
4. HANDLE hThread = (HANDLE)_beginthread(threadQueueEntry, 0, hIOCP);
   
5. =========================================================================
   
6. void threadQueueEntry(HANDLE hIOCP)
   
7. {
   
8.         threadmain(hIOCP);
   
9.         _endthread();
   
10. }
    
11. 
    
12. void threadmain(HANDLE hIOCP)
    
13. {
    
14.         std::list<std::string> lstString;
    
15.         DWORD dwTransferred = 0;
    
16.         ULONG_PTR CompletionKey = 0;                //这个是new出来的
    
17.         OVERLAPPED* pOverlapped;       
    
18.         while (GetQueuedCompletionStatus(hIOCP, &dwTransferred, &CompletionKey, &pOverlapped, INFINITE))
    
19.         {
    
20.                 IOCP_PARAM* pParam = (IOCP_PARAM*)CompletionKey;
    
21.                 if ((dwTransferred == 0) && (CompletionKey == NULL))
    
22.                 {
    
23.                         //delete pParam;                //这里是我的想法
    
24.                         break;
    
25.                 }
    
26.                 if (pParam->nOperator == IocpListPush)
    
27.                 {
    
28.                         lstString.push_back(pParam->strData);
    
29.                 }
    
30.                 else if (pParam->nOperator == IocpListPop)
    
31.                 {
    
32.                         std::string* pStr = NULL;
    
33.                         if (lstString.size() > 0)
    
34.                         {
    
35.                                 pStr = new std::string(lstString.front());
    
36.                                 lstString.pop_front();
    
37.                         }
    
38.                         if (pParam->cbFunc)
    
39.                         {
    
40.                                 pParam->cbFunc(pStr);
    
41.                         }
    
42.                 }
    
43.                 else if (pParam->nOperator == IocpListEmpty)
    
44.                 {
    
45.                         lstString.clear();
    
46.                 }
    
47.                 delete pParam;
    
48.         }
    
49.         //_endthread();
    
50. }
    
51. 
    

复制代码

我的理解是上面CompletionKey异常时,break直接结束循环没有调用到delete才会导致内存泄漏，所以在出口那里加上delete就可以了。
还有种说法是因为线程在执行_endthread()的时候结束了，所以来不及析构线程中的变量，导致内存泄漏。间接调用有作用域，所以出作用域就会全部析构掉里面的变量

最佳答案

月排行榜 / 总排行榜

lhgzbxhz

2022-2-11 10:34:59

2736946915 发表于 2022-2-10 19:20
23行没有的话，break会结束循环，而delete在循环尾部，不会执行到，这样CompletionKey的内存就得不到释放

但是它是NULL，为啥要delete？

跳转到最佳答案楼层

lhgzbxhz

其实线程函数返回时就会自动退出，不需要显式调用_endthread()函数
不过21行判断CompletionKey == NULL，20行把CompletionKey赋值给pParam，23行再delete，不就成了delete NULL吗？
按理来说，我没看出来这里有哪里可能会内存泄漏，你要不把完整代码发上来？

2736946915

lhgzbxhz 发表于 2022-2-9 12:12
其实线程函数返回时就会自动退出，不需要显式调用_endthread()函数
不过21行判断CompletionKey == NULL，2 ...

23行没有的话，break会结束循环，而delete在循环尾部，不会执行到，这样CompletionKey的内存就得不到释放

lhgzbxhz

2736946915 发表于 2022-2-10 19:20
23行没有的话，break会结束循环，而delete在循环尾部，不会执行到，这样CompletionKey的内存就得不到释放

但是它是NULL，为啥要delete？

2736946915

lhgzbxhz 发表于 2022-2-11 10:34
但是它是NULL，为啥要delete？

好像是了，原来还是endthread()析构太快导致内存泄漏，所以作用域限制，感谢