[PATCH 1/2] ieframe: Simplify IPersist_Load, make error handling more straightforward (Coverity)

Jacek Caban jacek at codeweavers.com
Mon Apr 11 10:30:35 CDT 2016


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.

Thanks,
Jacek



More information about the wine-devel mailing list