[已解决]内存泄漏，为啥cstring会造成内存泄漏，这看不明白

SyY

这个函数，这里是开始调用出内存泄漏的地方




void EnumNetCards()
{

        DWORD Status, Problem;
        LPTSTR Buffer = NULL;
        DWORD BufSize = 0;

        // 返回所有设备信息
        HDEVINFO hDevInfo = SetupDiGetClassDevs(NULL, NULL, 0, DIGCF_PRESENT | DIGCF_ALLCLASSES);

        if (INVALID_HANDLE_VALUE == hDevInfo)
                return;


        SP_DEVINFO_DATA DeviceInfoData = { sizeof(SP_DEVINFO_DATA) };


        //
        // 枚举设备
        //
        for (DWORD DeviceId = 0;
                SetupDiEnumDeviceInfo(hDevInfo, DeviceId, &DeviceInfoData);
                DeviceId++)
        {

                // 获得设备的状态
                if (CM_Get_DevNode_Status(&Status, &Problem, DeviceInfoData.DevInst, 0) != CR_SUCCESS)
                        continue;


                // 获取设备类名
                TCHAR szDevName[MAX_PATH] = _T("");
                if (GetRegistryProperty(hDevInfo, &DeviceInfoData, SPDRP_CLASS, &Buffer, (PULONG)&BufSize))
                {
                        lstrcpyn(szDevName, Buffer, MAX_PATH);
                }


                if (lstrcmp(szDevName, _T("Net")) == 0)
                {
                        TCHAR szName[MAX_PATH] = _T("");
                        if (GetRegistryProperty(hDevInfo, &DeviceInfoData, SPDRP_ENUMERATOR_NAME, &Buffer, (PULONG)&BufSize))
                        {
                                lstrcpyn(szName, Buffer, MAX_PATH);
                        }


                        if (lstrcmp(szName, _T("ROOT")) != 0)
                        {

                                if (GetRegistryProperty(hDevInfo, &DeviceInfoData, SPDRP_DRIVER, &Buffer, (PULONG)&BufSize))
                                {
                                        lstrcpyn(szName, Buffer, MAX_PATH);

                                         //获取设备描述
                                        if (GetRegistryProperty(hDevInfo, &DeviceInfoData, SPDRP_DEVICEDESC, &Buffer, (PULONG)&BufSize))
                                        {

                                                lstrcpyn(szName, Buffer, MAX_PATH);


                                                CString str1 = szName;
                                                CString str2 = _T("");
                                                str2 = str1.Mid(0, 11);
                                                if (str2 == "Realtek USB")
                                                {
                                                        if (bRet_server)
                                                        {
                                                                if (ControlDevice(DeviceId, hDevInfo))
                                                                {
                                                                        printf("Successful\n");
                                                                }
                                                                else
                                                                {
                                                                        printf("FAILED\n");
                                                                }
                                                        }
                                                }
                                        }
                                }
                        }
                }
        }

        SetupDiDestroyDeviceInfoList(hDevInfo);
}




这里是内存泄漏的代码块：
CStringData* CAfxStringMgr::Allocate( int nChars, int nCharSize ) throw()
{
        size_t nTotalSize;
        CStringData* pData;
        size_t nDataBytes;

        ASSERT(nCharSize > 0);
       
        if(nChars < 0)
        {
                ASSERT(FALSE);
                return NULL;
        }
       
        nDataBytes = (nChars+1)*nCharSize;
        nTotalSize = sizeof( CStringData )+nDataBytes;
        pData = (CStringData*)malloc( nTotalSize );
        if (pData == NULL)
                return NULL;
        pData->pStringMgr = this;
        pData->nRefs = 1;
        pData->nAllocLength = nChars;
        pData->nDataLength = 0;

        return pData;
}

最佳答案

月排行榜 / 总排行榜

tony666

2021-3-23 16:01:11

SyY 发表于 2021-3-23 13:42
我直接放弃用cstring了，把cstring作为全局变量也不行，cstring的free是每次都有进的，具体哪里问题确实 ...

你不是想禁用网卡么，直接com组件多省事。。
https://blog.csdn.net/qq_38777624/article/details/114930369

跳转到最佳答案楼层

SyY

CString str1 = szName;                                                 CString str2 = _T("");                                                 str2 = str1.Mid(0, 11);就是这一句，str2这里，发生的内存泄漏，到底是为什么呢？？？？？csrting不是不会内存泄漏吗

tony666

你的意思是崩溃？ 有没有可能str1 小于11

cpp2019

成对使用，成对使用，成对使用！
内存：
malloc出来的要free,
new出来的要delete,
文件、各类名柄：
open完了close.

SyY

tony666 发表于 2021-3-22 18:57
你的意思是崩溃？ 有没有可能str1 小于11

不是崩溃，只有检查内存泄漏才会停在这里，我发现，不是str2的问题，因为注释掉这一句也会停在上一句，就是貌似是szname的问题，会不会是tchar和lptstr 和cstring的转换哪里有问题啊？

SyY

cpp2019 发表于 2021-3-22 20:33
成对使用，成对使用，成对使用！
内存：
malloc出来的要free,

你说的没错，我没有new啊，cstring只是一个对象，但是转换前是一个lptstr的指针，可能这里有问题？

SyY

tony666 发表于 2021-3-22 18:57
你的意思是崩溃？ 有没有可能str1 小于11

                        TCHAR szName[MAX_PATH] = _T("");
                        if (GetRegistryProperty(hDevInfo, &DeviceInfoData, SPDRP_ENUMERATOR_NAME, &Buffer, (PULONG)&BufSize))
                        {
                                lstrcpyn(szName, Buffer, MAX_PATH);
                        }
是否会是这句有问题？

SyY

tony666 发表于 2021-3-22 18:57
你的意思是崩溃？ 有没有可能str1 小于11

BOOL GetRegistryProperty(HDEVINFO DeviceInfoSet,
        PSP_DEVINFO_DATA DeviceInfoData,
        ULONG Property,
        PVOID Buffer,
        PULONG Length)
{
        while (!SetupDiGetDeviceRegistryProperty(DeviceInfoSet,
                DeviceInfoData,
                Property,
                NULL,
                (BYTE *)*(TCHAR **)Buffer,
                *Length,
                Length))
        {
                // 长度不够则重新分配缓冲区
                if (GetLastError() == ERROR_INSUFFICIENT_BUFFER)
                {
                        if (*(LPTSTR *)Buffer)
                                LocalFree(*(LPTSTR *)Buffer);

                        *(LPTSTR *)Buffer = (PCHAR)LocalAlloc(LPTR, *Length);
                }
                else
                {
                        return false;
                }
        }

        return (BOOL)(*(LPTSTR *)Buffer)[0];
}
会不会是我 这个调用的函数 的问题啊

cpp2019

SyY 发表于 2021-3-23 09:07
TCHAR szName[MAX_PATH] = _T("");
                        if (GetRegistryProperty(hDevInfo, &DeviceInfoData, SPDRP_E ...

问题在于有malloc这段代码，你的代码中申请的堆内存用完了并没有释放，你的函数设计有问题！好好学学内存这块的基础！

SyY

cpp2019 发表于 2021-3-23 09:53
问题在于有malloc这段代码，你的代码中申请的堆内存用完了并没有释放，你的函数设计有问题！好好学学内 ...

   {
                // 长度不够则重新分配缓冲区
                if (GetLastError() == ERROR_INSUFFICIENT_BUFFER)
                {
                        if (*(LPTSTR *)Buffer)
                                LocalFree(*(LPTSTR *)Buffer);

                        *(LPTSTR *)Buffer = (PCHAR)LocalAlloc(LPTR, *Length);
                }
                else
                {
                        return false;
                }
        }

你说的是这里有问题？这里是释放了再申请，这有问题吗，每次进来都会释放掉

tony666

我只看出来最后退出EnumNetCards 的时候 最后一个buffer没释放。。。
另外请教个问题，你是用啥工具查看内存泄漏的，我从来都是靠眼。。。

cpp2019

8楼的函数也有问题，只有在if满足时才会调用一次free，1楼你自己标明的函数也有问题，变量作用域你没搞懂。

1. 这里是内存泄漏的代码块：
   
2. CStringData* CAfxStringMgr::Allocate( int nChars, int nCharSize ) throw()
   
3. {
   
4.         size_t nTotalSize;
   
5.         CStringData* pData;
   
6.         size_t nDataBytes;
   
7. 
   
8.         ASSERT(nCharSize > 0);
   
9.       
   
10.         if(nChars < 0)
    
11.         {
    
12.                 ASSERT(FALSE);
    
13.                 return NULL;
    
14.         }
    
15.       
    
16.         nDataBytes = (nChars+1)*nCharSize;
    
17.         nTotalSize = sizeof( CStringData )+nDataBytes;
    
18.         pData = (CStringData*)malloc( nTotalSize );
    
19.         if (pData == NULL)
    
20.                 return NULL;
    
21.         pData->pStringMgr = this;
    
22.         pData->nRefs = 1;
    
23.         pData->nAllocLength = nChars;
    
24.         pData->nDataLength = 0;
    
25. 
    
26.         return pData;
    
27. }

复制代码

SyY

tony666 发表于 2021-3-23 12:14
我只看出来最后退出EnumNetCards 的时候 最后一个buffer没释放。。。
另外请教个问题，你是用啥工具查看内 ...

我放弃用cstring了 然后就没有内存泄漏了 直接用auto类型
内存泄漏用        //_CrtDumpMemoryLeaks();关键字

SyY

cpp2019 发表于 2021-3-23 13:12
8楼的函数也有问题，只有在if满足时才会调用一次free，1楼你自己标明的函数也有问题，变量作用域你没搞懂。 ...

我直接放弃用cstring了，把cstring作为全局变量也不行，cstring的free是每次都有进的，具体哪里问题确实找不到，总体就是 tchar和cstring的转换  这里可能有编码的问题，中文的时候会报错

tony666

SyY 发表于 2021-3-23 13:42
我直接放弃用cstring了，把cstring作为全局变量也不行，cstring的free是每次都有进的，具体哪里问题确实 ...

你不是想禁用网卡么，直接com组件多省事。。
https://blog.csdn.net/qq_38777624/article/details/114930369

tony666

SyY 发表于 2021-3-23 13:41
我放弃用cstring了 然后就没有内存泄漏了 直接用auto类型
内存泄漏用        //_CrtDumpMemoryLeaks();关键字

学习了，谢谢

SyY

tony666 发表于 2021-3-23 16:01
你不是想禁用网卡么，直接com组件多省事。。
https://blog.csdn.net/qq_38777624/article/details/11493 ...

确实可以，这个还挺好用，com组件好东西