自己类中的C ++ GDI +内存泄漏

编码器

我正在此代码中搜索内存泄漏。我是GDI +的新手,我不确定自己做错了什么。您在下面看到的类在我的主函数中被循环调用。每次循环迭代,我都会将另一个向量推入函数。一切正常,除了内存泄漏。我尝试了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;
}
乔纳森·波特

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);可释放它。

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

本文收集自互联网,转载请注明来源。

如有侵权,请联系 [email protected] 删除。

编辑于
0

我来说两句

0 条评论
登录 后参与评论

相关文章