Fix possible memory leak in wxICOHandler loading code.

Use wxScopedArray<> instead of a raw pointer to ensure that the memory is
always freed, even in case of error return.

See #15918.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@76128 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin
2014-03-12 23:03:55 +00:00
parent 2c64172825
commit d8090867aa

View File

@@ -1419,8 +1419,8 @@ bool wxICOHandler::DoLoadFile(wxImage *image, wxInputStream& stream,
wxUint16 nType = wxUINT16_SWAP_ON_BE(IconDir.idType);
// loop round the icons and choose the best one:
ICONDIRENTRY *pIconDirEntry = new ICONDIRENTRY[nIcons];
ICONDIRENTRY *pCurrentEntry = pIconDirEntry;
wxScopedArray<ICONDIRENTRY> pIconDirEntry(nIcons);
ICONDIRENTRY *pCurrentEntry = pIconDirEntry.get();
int wMax = 0;
int colmax = 0;
int iSel = wxNOT_FOUND;
@@ -1467,7 +1467,7 @@ bool wxICOHandler::DoLoadFile(wxImage *image, wxInputStream& stream,
else
{
// seek to selected icon:
pCurrentEntry = pIconDirEntry + iSel;
pCurrentEntry = pIconDirEntry.get() + iSel;
// NOTE: seeking a positive amount in wxFromCurrent mode allows us to
// load even non-seekable streams (see wxInputStream::SeekI docs)!
@@ -1485,8 +1485,6 @@ bool wxICOHandler::DoLoadFile(wxImage *image, wxInputStream& stream,
}
}
delete [] pIconDirEntry;
return bResult;
}