C++ GDI+ 自己的内存泄漏 class

Question

我正在寻找这段代码中的内存泄漏。我是 GDI+ 的新手，我不确定自己做错了什么。您在下面看到的 class 在我的主函数中循环调用。每次循环迭代我都会将另一个向量推送到该函数。除了内存泄漏外，一切正常。我尝试了程序 cppCheck 来查找泄漏，但没有发现内存泄漏：/ 我解决这个问题的最后机会是请教比我更有 GDI+

经验的人

非常感谢您的帮助，抱歉代码太长:)

#include "helper.h"

Gui::Gui(const TCHAR* fileName) {
    this->fileName = fileName;
}

void Gui::drawGui(Gdiplus::Bitmap* image, std::vector<std::wstring> &vec) {

    // Init graphics
    Gdiplus::Graphics* graphics = Gdiplus::Graphics::FromImage(image);

    Gdiplus::Pen penWhite (Gdiplus::Color::White);
    Gdiplus::Pen penRed   (Gdiplus::Color::Red);
    Gdiplus::SolidBrush redBrush(Gdiplus::Color(255, 255, 0, 0));
    penRed.SetWidth(8);

    unsigned short marginTop = 15;
    unsigned short marginLeft = 5;
    unsigned short horizontalBarsizeStart = marginLeft + 60;


    for (unsigned short iter = 0; iter < 8; iter++) {
        // Draw text
        std::wstring coreLabel = L"Core " + std::to_wstring(iter) + L':';
        Gdiplus::Font myFont(L"Arial", 12);
        Gdiplus::PointF origin(marginLeft, marginTop - 10);
        graphics->DrawString(coreLabel.c_str(), coreLabel.length(), &myFont, origin, &redBrush);

        // Draw CPU lines
        unsigned short horizontalBarsizeEnd = horizontalBarsizeStart + std::stoi(vec.at(iter)); // 100 == Max cpu load
        graphics->DrawLine(&penRed, horizontalBarsizeStart, marginTop, horizontalBarsizeEnd, marginTop);

        // Draw border
        Gdiplus::Rect rect(horizontalBarsizeStart, marginTop - 5, 100, 8);
        graphics->DrawRectangle(&penWhite, rect);

        // Next element
        marginTop += 17;
    }
}


bool Gui::SetColorBackgroundFromFile(std::vector<std::wstring> &vec) {

    Gdiplus::GdiplusStartupInput gdiplusStartupInput;
    ULONG_PTR gdiplusToken;
    // Initialize GDI+.
    Gdiplus::GdiplusStartup(&gdiplusToken, &gdiplusStartupInput, NULL);

    HDC hdc = GetDC(NULL);

    // Load the image. Any of the following formats are supported: BMP, GIF, JPEG, PNG, TIFF, Exif, WMF, and EMF
    Gdiplus::Bitmap* image = Gdiplus::Bitmap::FromFile(this->fileName, false);

    if (image == NULL) {
        return false;
    }

    // Draw the gui
    this->drawGui(image, vec);

    // Get the bitmap handle
    HBITMAP hBitmap = NULL;
    Gdiplus::Status status = image->GetHBITMAP(RGB(0, 0, 0), &hBitmap);
    if (status != Gdiplus::Ok) {
        return false;
    }

    BITMAPINFO bitmapInfo = { 0 };
    bitmapInfo.bmiHeader.biSize = sizeof(BITMAPINFOHEADER);

    // Check what we got
    int ret = GetDIBits(hdc, hBitmap, 0, 0, NULL, &bitmapInfo, DIB_RGB_COLORS);

    if (LOGI_LCD_COLOR_WIDTH != bitmapInfo.bmiHeader.biWidth || LOGI_LCD_COLOR_HEIGHT != bitmapInfo.bmiHeader.biHeight) {
        std::cout << "Oooops. Make sure to use a 320 by 240 image for color background." << std::endl;
        return false;
    }

    bitmapInfo.bmiHeader.biCompression = BI_RGB;
    bitmapInfo.bmiHeader.biHeight = -bitmapInfo.bmiHeader.biHeight; // this value needs to be inverted, or else image will show up upside/down

    BYTE byteBitmap[LOGI_LCD_COLOR_WIDTH * LOGI_LCD_COLOR_HEIGHT * 4]; // we have 32 bits per pixel, or 4 bytes

    // Gets the "bits" from the bitmap and copies them into a buffer 
    // which is pointed to by byteBitmap.
    ret = GetDIBits(hdc, hBitmap, 0,
    -bitmapInfo.bmiHeader.biHeight, // height here needs to be positive. Since we made it negative previously, let's reverse it again.
    &byteBitmap,
    (BITMAPINFO *)&bitmapInfo, DIB_RGB_COLORS);

    LogiLcdColorSetBackground(byteBitmap); // Send image to LCD

    // delete the image when done 
    if (image) {
        delete image;
        image = NULL;
        Gdiplus::GdiplusShutdown(gdiplusToken); // Shutdown GDI+
    }
return true;
}

Answer 1

在 drawGui() 中，您正在泄漏 graphics 对象。此行创建一个新的 Gdiplus::Graphics 对象：

Gdiplus::Graphics* graphics = Gdiplus::Graphics::FromImage(image);

但是一旦你完成了它，你就不会在任何地方调用 delete graphics 来删除它。

在 SetColorBackgroundFromFile 中，您正在泄漏 DC。

HDC hdc = GetDC(NULL);

这为屏幕获取了一个 DC，但是您没有调用 ReleaseDC(NULL, hdc); 来释放它。

在同一个函数中，您正在使用以下调用创建一个 HBITMAP：

Gdiplus::Status status = image->GetHBITMAP(RGB(0, 0, 0), &hBitmap);

但是你没有在任何地方调用 DeleteObject(hBitmap); 来释放它。

您还有一个问题，即如果出现错误，您的代码可以 return 而无需进行必要的清理。例如。如果 GetHBITMAP 调用失败，您会立即 return 并泄漏您在上面几行创建的 image 对象。

C++ GDI+ 自己的内存泄漏 class

C++ GDI+ memory leak in own class

c++

memory

gdi+

memory-leaks