Где утечка памяти из моего hdc / hbitmap? - PullRequest
Новая
       109

Где утечка памяти из моего hdc / hbitmap?

0 голосов
/ 29 октября 2018

поэтому я заметил, что часть моего кода теряет много памяти, когда он вызывается, и я пытался выяснить, где или почему он протекает, но я зашел в тупик.

Я пробовал отладчик Visual Studio 2017, чтобы сделать снимки, чтобы выяснить, где происходит утечка, но в соответствии с этим нет никаких серьезных утечек. Я также попробовал Deleaker, который я однажды заработал, который сказал мне, что я пропустил HDC и HBITMAP, но не мог сказать мне, сколько памяти.

Первая функция - это GetScreenBmp, где может быть утечка, но разве я не выпускаю все правильно? Я знаю, что не удаляю hBitmap, но мне нужно это вернуть. Это где утечка памяти? 

HBITMAP GetScreenBmp(HDC hdc, int screenPositionX, int screenPositionY, int screenSizeX, int screenSizeY) {
// Get screen dimensions
int nScreenWidth = GetSystemMetrics(SM_CXSCREEN);
int nScreenHeight = GetSystemMetrics(SM_CYSCREEN);
int nMousePositionX = 0, nMousePositionY = 0;

// Create compatible DC, create a compatible bitmap and copy the screen using BitBlt()
HDC hCaptureDC = CreateCompatibleDC(hdc);
HBITMAP hBitmap = CreateCompatibleBitmap(hdc, screenSizeX, screenSizeY);
HGDIOBJ hOld = SelectObject(hCaptureDC, hBitmap);
BOOL bOK = BitBlt(hCaptureDC, 0, 0, screenSizeX, screenSizeY, hdc, screenPositionX, screenPositionY, SRCCOPY | CAPTUREBLT);

SelectObject(hCaptureDC, hOld); // always select the previously selected object once done
DeleteObject(hOld);
DeleteDC(hCaptureDC);

return hBitmap;

Вторая часть - это фрагмент кода, в котором я не совсем уверен, правильно ли я удаляю все. 

HDC hdc = GetDC(0);
    HBITMAP hBitmap = GetScreenBmp(hdc, currentSplitInformationArray.screenPositionX, currentSplitInformationArray.screenPositionY, currentSplitInformationArray.screenSizeX, currentSplitInformationArray.screenSizeY);
    BITMAPINFO MyBMInfo = { 0 };
    MyBMInfo.bmiHeader.biSize = sizeof(MyBMInfo.bmiHeader);

    // Get the BITMAPINFO structure from the bitmap
    if (0 == GetDIBits(hdc, hBitmap, 0, 0, NULL, &MyBMInfo, DIB_RGB_COLORS)) {
        MessageBox(NULL, "Resource not available\nDo you want to try again?", "Account Details", MB_ICONWARNING | MB_CANCELTRYCONTINUE | MB_DEFBUTTON2);
    }

    // create the bitmap buffer
    BYTE* lpPixels = new BYTE[MyBMInfo.bmiHeader.biSizeImage];

    // Better do this here - the original bitmap might have BI_BITFILEDS, which makes it
    // necessary to read the color table - you might not want this.
    MyBMInfo.bmiHeader.biCompression = BI_RGB;
    MyBMInfo.bmiHeader.biHeight = currentSplitInformationArray.screenSizeY * -1;

    // get the actual bitmap buffer
    if (0 == GetDIBits(hdc, hBitmap, 0, currentSplitInformationArray.screenSizeY, (LPVOID)lpPixels, &MyBMInfo, DIB_RGB_COLORS)) {
        MessageBox(NULL, "Resource not available\nDo you want to try again?", "Account Details", MB_ICONWARNING | MB_CANCELTRYCONTINUE | MB_DEFBUTTON2);
    }

    ::SendMessage(testingComparison, STM_SETIMAGE,
        (WPARAM)IMAGE_BITMAP, (LPARAM)hBitmap);

    DeleteObject(&MyBMInfo);
    DeleteObject(hBitmap);
    ReleaseDC(NULL, hdc);
    delete[] lpPixels;

Заранее прошу прощения, если на этот вопрос уже был дан ответ, или если ответ легко найти, но я несколько часов пытался его исправить.

Ответы [ 2 ]

0 голосов
/ 29 октября 2018

Хорошо, я нашел решение. Сообщение STM_SETIMAGE возвращает предыдущее изображение, и вы должны обработать его самостоятельно. https://docs.microsoft.com/en-us/windows/desktop/Controls/stm-setimage

Вероятно, мне следует научиться лучше читать документацию в следующий раз, извините, что тратит время на все это. 

::SendMessage(testingComparison, STM_SETIMAGE,
            (WPARAM)IMAGE_BITMAP, (LPARAM)hBitmap);

Исправлено, просто делая 

HBITMAP oldBitmap = (HBITMAP)::SendMessage(testingComparison, STM_SETIMAGE,
        (WPARAM)IMAGE_BITMAP, (LPARAM)hBitmap);
    DeleteObject(oldBitmap);
0 голосов
/ 29 октября 2018

Используйте инструменты для отслеживания ваших утечек / распределений (кстати, вы не опубликовали, как вы обнаружили утечки в первую очередь).

Поскольку вы используете Visual Studio C ++, вы можете использовать встроенные инструменты. По сути, комбинация этих 3-х строк может сделать работу. 

    _CrtSetDbgFlag ( _CRTDBG_ALLOC_MEM_DF | _CRTDBG_LEAK_CHECK_DF );//to enable, safe to always set somewhere around program startup - on normal exit this will print whatever you leaked

    //_CrtDumpMemoryLeaks();//Dumps to see what has already been allocated

    //_CrtSetBreakAlloc(#number);//Use this to set breakpoint using the allocation number from heap dump to see where allocation takes place. If allocation happends before this line it will not work.
...