当前位置: 代码迷 >> VC/MFC >> 向子线程传接CString造成的内存泄露,求解
  详细解决方案

向子线程传接CString造成的内存泄露,求解

热度:51   发布时间:2016-05-02 03:41:59.0
向子线程传递CString造成的内存泄露,求解~
本帖最后由 Cacar 于 2015-09-16 14:58:04 编辑
过程如下:
(1)构造函数中得到本机IP地址:
CString CRecToolBoxDlg::GetIP()
{
WORD wVersionRequested;
WSADATA wsaData;
char name[255];    // 本机主机名
CString strIP;      // IP地址字符串
PHOSTENT hostinfo;  
wVersionRequested=MAKEWORD(2,0);   //winsock Dll
if(WSAStartup(wVersionRequested,&wsaData)==0)

if(gethostname(name,sizeof(name))==0)  //本地主机名
{
if((hostinfo=gethostbyname(name))!=NULL) //主机信息
{
// 在使能“监听网络”后,此处会有一个44byte的内存泄露(IP地址字符长度2*(13+1)+CString自己的28=44),原因未明
strIP = inet_ntoa(*(struct in_addr *)*hostinfo->h_addr_list); //IP地址

}
else
strIP = "IP检测失败";   
}  
WSACleanup();   // ??winsock DLL???
}

return strIP;  //返回本机IP
}

(2)主线程开始工作后,创建如下两个线程:

void CRecToolBoxDlg::OnCheckNet() 
{
CWnd *pWnd_text = GetDlgItem(IDC_STATIC_NETTEXT);
CWnd *pWnd_edit = GetDlgItem(IDC_EDIT_PORT);
CWnd *pWnd_btn = GetDlgItem(IDC_BTN_SETNET);


if(!m_bIsNetworkSet) // 若未设置网络参数,或监听未建立,则建立监听
{
if(pWnd_btn && pWnd_edit && pWnd_text) // 确保窗口已创建
{
pWnd_text->EnableWindow(TRUE);
//pWnd_edit->EnableWindow(TRUE);
pWnd_btn->EnableWindow(TRUE);

DWORD threadID,threadID2;

// 创建发送套接字
CString strIP = m_strIP;
m_sockSend.Create(m_uPortSend,SOCK_DGRAM,strIP);
m_sockSend.Bind(m_uPortSend,strIP);

// 创建监听套接字
//int iResult = m_sockListen.Create(m_uPort,SOCK_DGRAM,strIP);
//m_hSockListen = m_sockListen.Detach();
// 建立监听线程
m_hThreadListen = CreateThread((LPSECURITY_ATTRIBUTES)NULL,0,(LPTHREAD_START_ROUTINE)NetThreadProc,(LPVOID)this,CREATE_SUSPENDED,&threadID);
m_hThreadProcess = CreateThread((LPSECURITY_ATTRIBUTES)NULL,0,(LPTHREAD_START_ROUTINE)ProcessThreadProc,(LPVOID)this,CREATE_SUSPENDED,&threadID2);
// 开启监听线程
if(m_hThreadListen == NULL)
{
CloseHandle(m_hThreadListen); // 关闭监听线程
TRACE("创建监听线程失败!");
return;
}
else
{
// 启动监听线程
m_bIsNetworkSet = TRUE;
ResumeThread(m_hThreadListen);
}

// 开启数据处理线程
if(m_hThreadProcess == NULL)
{
CloseHandle(m_hThreadProcess);
TRACE("创建数据处理线程失败!");
return;
}
else
{
ResumeThread(m_hThreadProcess); // 启动数据处理线程
m_bStartProcess = TRUE;
}
}

}
else // 若监听已建立,则关闭监听
{
if(pWnd_btn && pWnd_edit && pWnd_text) // 确保窗口已创建
{
pWnd_text->EnableWindow(FALSE);
pWnd_btn->EnableWindow(FALSE);

//m_sockListen.Attach(m_hSockListen);
//m_sockListen.Close();

m_bIsNetworkSet = FALSE;

m_sockSend.Close();
TRACE("UDP连接已关闭.....\r\n");
}
}

}

两个线程中都是通过调用父窗口指针传递成员m_strIP的,似乎没什么问题,为什么会有内存泄露呢?定位在这一句:
strIP = inet_ntoa(*(struct in_addr *)*hostinfo->h_addr_list); 

但不调用OnCheckNet()的话,strIP是可以被正确释放的。不明白OnCheckNet()调用与否为什么会影响到GetIP()中strIP的释放~求大神指点
------解决思路----------------------
有时不将“调用函数名字+各参数值,进入函数后各参数值,中间变量值,退出函数前准备返回的值,返回函数到调用处后函数名字+各参数值+返回值”这些信息写日志到文件中是无论如何也发现不了问题在哪里的,包括捕获各种异常、写日志到屏幕、单步或设断点或生成core文件、……这些方法都不行! 写日志到文件参考下面:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#ifdef WIN32
    #include <windows.h>
    #include <io.h>
#else
    #include <unistd.h>
    #include <sys/time.h>
    #include <pthread.h>
    #define  CRITICAL_SECTION   pthread_mutex_t
    #define  _vsnprintf         vsnprintf
#endif
//Log{
#define MAXLOGSIZE 20000000
#define MAXLINSIZE 16000
#include <time.h>
#include <sys/timeb.h>
#include <stdarg.h>
char logfilename1[]="MyLog1.log";
char logfilename2[]="MyLog2.log";
static char logstr[MAXLINSIZE+1];
char datestr[16];
char timestr[16];
char mss[4];
CRITICAL_SECTION cs_log;
FILE *flog;
#ifdef WIN32
void Lock(CRITICAL_SECTION *l) {
    EnterCriticalSection(l);
}
void Unlock(CRITICAL_SECTION *l) {
    LeaveCriticalSection(l);
}
#else
void Lock(CRITICAL_SECTION *l) {
    pthread_mutex_lock(l);
}
void Unlock(CRITICAL_SECTION *l) {
    pthread_mutex_unlock(l);
}
#endif
void LogV(const char *pszFmt,va_list argp) {
    struct tm *now;
    struct timeb tb;

    if (NULL==pszFmt
------解决思路----------------------
0==pszFmt[0]) return;
    _vsnprintf(logstr,MAXLINSIZE,pszFmt,argp);
    ftime(&tb);
    now=localtime(&tb.time);
    sprintf(datestr,"%04d-%02d-%02d",now->tm_year+1900,now->tm_mon+1,now->tm_mday);
    sprintf(timestr,"%02d:%02d:%02d",now->tm_hour     ,now->tm_min  ,now->tm_sec );
    sprintf(mss,"%03d",tb.millitm);
    printf("%s %s.%s %s",datestr,timestr,mss,logstr);
    flog=fopen(logfilename1,"a");
    if (NULL!=flog) {
        fprintf(flog,"%s %s.%s %s",datestr,timestr,mss,logstr);
        if (ftell(flog)>MAXLOGSIZE) {
            fclose(flog);
            if (rename(logfilename1,logfilename2)) {
                remove(logfilename2);
                rename(logfilename1,logfilename2);
            }
        } else {
            fclose(flog);
        }
    }
}
void Log(const char *pszFmt,...) {
    va_list argp;

    Lock(&cs_log);
    va_start(argp,pszFmt);
    LogV(pszFmt,argp);
    va_end(argp);
    Unlock(&cs_log);
}
//Log}
int main(int argc,char * argv[]) {
    int i;
#ifdef WIN32
    InitializeCriticalSection(&cs_log);
#else
    pthread_mutex_init(&cs_log,NULL);
#endif
    for (i=0;i<10000;i++) {
        Log("This is a Log %04d from FILE:%s LINE:%d\n",i, __FILE__, __LINE__);
    }
#ifdef WIN32
    DeleteCriticalSection(&cs_log);
#else
    pthread_mutex_destroy(&cs_log);
#endif
    return 0;
}
//1-78行添加到你带main的.c或.cpp的那个文件的最前面
//81-85行添加到你的main函数开头
//89-93行添加到你的main函数结束前
//在要写LOG的地方仿照第87行的写法写LOG到文件MyLog1.log中

------解决思路----------------------
要看看你的API调用是否有正确释放。线程中的资源是否都对应释放了
------解决思路----------------------
CString  使用的是引用计数的方式

m_strIP = GetIP();
CString strIP = m_strIP;
m_sockSend.Create(m_uPortSend,SOCK_DGRAM,strIP);
m_sockSend.Bind(m_uPortSend,strIP);

以上代码用的其实都是同一份内存数据
你说OnCheckNet 会影响释放, 你检查下上面的 Create, Bind 函数, 看是有有问题


------解决思路----------------------
inet_ntoa返回的是一个静态字符数组,你最好使用数组去memcpy

不过使用CString理论与逻辑上也说的过去,你在使用这个CString的时候GetBuffer了没?
  相关解决方案