Ошибка LoadBitmap после нескольких вызовов

У вашего кода есть несколько проблем, некоторые второстепенные, другие фатальные (и реализация действительно только работает, потому что ОС готова справиться с этими распространенными ошибками). Вот аннотированный список оригинального кода:

void CDLP_Printer_ControlDlg::DisplayBMPfromSVG(CString& strDsiplayFile) {
//                                              ^ should be const CString&
HBITMAP hbmp_temp = (HBITMAP)::LoadImage(0, strDsiplayFile, IMAGE_BITMAP, 0, 0,
LR_LOADFROMFILE);
if (!hbmp_temp) {
//hbmp_temp = ::LoadBitmap(AfxGetInstanceHandle(), MAKEINTRESOURCE(IDB_BITMAP1));
ActionList.AddString(L"Bitmap Load Failure: GetBMPromSVG");
ActionList.UpdateWindow();
if (!hbmp_temp)
//      You already know, that the condition is true (unless your commented out code
//      is supposed to run).
return;
}

CBitmap bmp_temp;
bmp_temp.Attach(hbmp_temp);
//  ^ This should immediately follow the LoadImage call, to benefit from automatic
//    resource management. (What's the point of using MFC when you decide to implement
//    manual resource management on top of it?)
mProjectorWindow.m_picControl.ModifyStyle(0xF, SS_BITMAP, SWP_NOSIZE);
//                                            ^ Use named constants. No one is going to
//                                              look up the documentation just to find
//                                              out, what you are trying to do.
mProjectorWindow.m_picControl.SetBitmap(bmp_temp);
//  The GDI object (hbmp_temp) now has two owners, the CBitmap instance bmp_temp, and
//  the picture control. At the same time, you are throwing away the handle previously
//  owned by the control. This is your GDI resource leak.

return;
//  ^ Superfluous. This is merely confusing readers. Remove it.
}
// This is where things go fatal: The bmp_temp d'tor runs, destroying the GDI resource
// hbmp_temp, that's also owned by the control. This should really blow up in your face
// but the OS knows that developers cannot be trusted anymore, and covers your ass.

Две основные проблемы:

• Ошибка удаления ресурсов GDI, принадлежащих вашему коду (возвращаемое значение SetBitmap). Это в конечном итоге приводит к неудаче любой попытки создать дополнительные ресурсы GDI.
• Висячий указатель (HBITMAP) вызвано назначением двух владельцев одному и тому же ресурсу (hbmp_temp).
• Здесь действительно другая проблема. Поскольку вы назначили двух владельцев для растрового ресурса, это приведет к двойному удалению. Это не так, потому что вы выбрали очистку ресурса. (Тот факт, что вы не знаете, что делаете, спас вас здесь. Помните об этом в следующий раз, когда вы отпраздновать свой «сможет сделать» отношение.)

Ниже приведена версия с исправленным управлением ресурсами. Это не имеет никакого смысла для вас, поскольку вы недостаточно хорошо знаете MFC (или C ++), чтобы понять, как один из них помогает в автоматическом управлении ресурсами. Так или иначе:

void CDLP_Printer_ControlDlg::DisplayBMPfromSVG(CString& strDsiplayFile) {
HBITMAP hbmp_temp = (HBITMAP)::LoadImage(0, strDsiplayFile, IMAGE_BITMAP, 0, 0,
LR_LOADFROMFILE);
// Immediately attach a C++ object, so that resources will get cleaned up
// regardless how the function is exited.
CBitmap bmp_temp;
if (!bmp_temp.Attach(hbmp_temp)) {
// Log error/load placeholder image
return;
}

mProjectorWindow.m_picControl.ModifyStyle(0xF, SS_BITMAP, SWP_NOSIZE);
// Swap the owned resource of bmp_temp with that of the control:
bmp_temp.Attach(mProjectorWindow.m_picControl.SetBitmap(bmp_temp.Detach()));
}

Последняя строка является критической частью. Он реализует канонический способ обмена необработанными ресурсами Windows API с оболочками управления ресурсами. Это последовательность операций:

1. bmp_temp.Detach() освобождает право собственности на ресурс GDI.
2. SetBitmap() передает владение ресурсом GDI элементу управления и возвращает предыдущий объект GDI (если есть).
3. bmp_temp.Attach() приобретает право собственности на возвращенный ресурс GDI. Это гарантирует, что предыдущий ресурс будет очищен, когда bmp_temp выходит за рамки (в конце функции).