[PATCH 1/2] ieframe: Simplify IPersist_Load, make error handling more straightforward (Coverity)
bunglehead at gmail.com
Mon Apr 11 11:00:52 CDT 2016
On 11.04.2016 18:30, Jacek Caban wrote:
> Hi Nikolay,
> On 04/08/16 21:56, Nikolay Sivov wrote:
>> + /* Now we're going to read in the iconfile and iconindex.
>> + If we don't find them, that's not a failure case -- it's possible
>> + that they just aren't in there. */
>> + hr = IPropertySetStorage_Open(This->property_set_storage, &FMTID_Intshcut,
>> + STGM_READWRITE | STGM_SHARE_EXCLUSIVE, &pPropStg);
> It's an already existing bug, but we should handle Open failure here.
I think I kept it intentionally, to keep original logic intact in
cleanup patch. By handling it you mean making Load() fail if Open()
fails? Do you want me to make this change in the same patch? I can make
it 3/3 too.
More information about the wine-devel