msi: Uninitialized variable fix (Coverity)
James Hawkins
truiken at gmail.com
Tue Sep 9 16:30:10 CDT 2008
On Tue, Sep 9, 2008 at 4:24 PM, Andrew Talbot
<Andrew.Talbot at talbotville.com> wrote:
> James Hawkins wrote:
>
>> On Tue, Sep 9, 2008 at 3:54 PM, Andrew Talbot
>> <andrew.talbot at talbotville.com> wrote:
>>> Fix for Coverity error CID: 762.
>>>
>>> [...]
>>> - RegCloseKey(userdata);
>>> + if (userdata) RegCloseKey(userdata);
>>> return rc;
>>
>> Please don't add another NULL-before-free check.
>>
>
> Hi James,
>
> Sorry, I don't understand what I have done wrong. RegCloseKey() will return ERROR_INVALID_HANDLE if called with hkey==NULL.
>
...and we don't care what value it returns.
--
James Hawkins
More information about the wine-devel
mailing list