OLEAUT32: Use the FAILED macro to check for failure of interface
methods.
Robert Shearman
rob at codeweavers.com
Sat Dec 31 04:54:50 CST 2005
Marcus Meissner wrote:
>On Sat, Dec 31, 2005 at 11:12:33AM +0900, Mike McCormack wrote:
>
>
>>Robert Shearman wrote:
>>
>>
>>
>>>These changes are wrong. IStream_Read can return S_FALSE, which means
>>>that the call was successful, but didn't do what was intended. From MSDN:
>>>
>>>S_FALSE
>>> The data cannot be read from the stream object. Depending on the
>>> implementation, either S_FALSE or an error code could be returned
>>> when reading past the end of the stream.
>>>
>>>
>>Fair enough. Then perhaps the ole storage code is broken, because it
>>returns S_FALSE when it correctly read the requested amount of data, and
>>IOlePicture_Load was failing to read a bitmap from an MSI package.
>>
>>
>
>Do you have a testcase installer?
>
No need. Working with Mike, I've found the bug and I'll send a patch soon.
--
Rob Shearman
More information about the wine-devel
mailing list