HDC内存泄漏(发布HDC /删除hdc)

时间:2016-07-29 04:21:37

标签: c++ memory-management memory-leaks hdc

我遇到此HDC内存泄漏问题。你们可以检查我是否正确地发布/删除了HDC?

谢谢!

BITMAP bm;
HBITMAP hbmap;
HBITMAP hBitmapOld;
BITMAPINFO bmi;
HDC hdcShot;

...

while(true)
{
    if (!TakeScreenshot(GameWindow, bm, hbmap, bmi, hdcShot, hBitmapOld, appWnd))
                    break;

        HBITMAP hbmapNew = CreateCompatibleBitmap(hdcShot, rcWindow.right - rcWindow.left, rcWindow.bottom - rcWindow.top);

        HDC hdcShotNew = CreateCompatibleDC(hdcShot);

        HBITMAP OldBmp = (HBITMAP)SelectObject(hdcShotNew, hbmapNew);


        BitBlt(hdcShotNew, 0, 0, rcWindow.right - rcWindow.left/*Window WIDTH*/, rcWindow.bottom - rcWindow.top/*Window HEIGHT*/
            , hdcShot, 0, 0, SRCCOPY);


        pPixels = new RGBQUAD[bm.bmWidth * bm.bmHeight];
        if (!pPixels) return false;

        SelectObject(hdcShotNew, OldBmp);


        if (!GetDIBits(hdcShotNew, hbmapNew, 0, bm.bmHeight, pPixels, &bmi, DIB_RGB_COLORS))
        {
            DeleteDC(hdcShot);
            delete[] pPixels;

            return false;
        }



// dont mind about this
        ScanContents scanContentsMain(bm, rcWindow, pPixels);
// dont mind about this
        ScanBMPHorizontal(&scanContentsMain);




        //free memory - I might have cleared the memory incorrectly here! Please check!
        free(pPixels);
        SelectObject(hdcShot, hBitmapOld);
                DeleteDC(hdcShot);
                DeleteObject(hbmapNew);
                DeleteDC(hdcShotNew);
}

TakeScreenShot Func(不是很重要,但它显示了一些变量的初始化)

    bool TakeScreenshot(std::string WindowToFind, BITMAP &bm, HBITMAP &hbmap, BITMAPINFO &bmi, HDC &hdcShot, HBITMAP &hBitmapOld, HWND &hwnd)
{
    RECT rc;
    GetWindowRect(hwnd, &rc);
    hdcShot = CreateCompatibleDC(0);
    hbmap = CreateCompatibleBitmap(GetDC(0), rc.right - rc.left/*Window WIDTH*/, rc.bottom - rc.top/*Window HEIGHT*/);

    SelectObject(hdcShot, hbmap);


    BitBlt(hdcShot, 0, 0, rc.right - rc.left/*Window WIDTH*/, rc.bottom - rc.top/*Window HEIGHT*/
        , GetDC(0), rc.left, rc.top, SRCCOPY);

//Ignore this
    if (!GetObject(hbmap, sizeof(BITMAP), (LPSTR)&bm))
        return false;
    int bitsPerPixel = bm.bmBitsPixel;


 //Ignore this  
    if (bitsPerPixel != 32 || bm.bmPlanes != 1)
        return false;

//Don't mind about this too much 
    SetupBitmapInfo(bmi, bm.bmWidth, bm.bmHeight, bitsPerPixel);


    return true;

}

我检查了删除器,发现我正在努力解决HDC漏洞问题。我不确定我做错了什么。

1 个答案:

答案 0 :(得分:4)

这里有资源泄漏:

hbmap = CreateCompatibleBitmap(GetDC(0), rc.right - rc.left, rc.bottom - rc.top);

你应该改为

HDC hdc = GetDC(0);
CreateCompatibleBitmap(hdc, rc.right - rc.left, rc.bottom - rc.top);
... 
ReleaseDC(0, hdc);//call this before exiting the function

free(pPixels)是错误的(虽然在这种情况下仍然可以清理)。将free(pPixels)替换为delete[]pPixels

改变这个:

SelectObject(hdcShot, hBitmapOld);
DeleteDC(hdcShot);
DeleteObject(hbmapNew);
DeleteDC(hdcShotNew);

对此:

SelectObject(hdcShot, OldBmp);
DeleteObject(hbmapNew);
DeleteObject(hBitmapOld);
DeleteDC(hdcShot);
DeleteDC(hdcShotNew);