quartz: Use proper alloc/free functions for COM objects

Chris Robinson chris.kcat at gmail.com
Thu Mar 8 19:53:14 CST 2007


On Thursday 08 March 2007 08:14:54 am you wrote:
> While reading your Patch, I see the above Places, where the result
> from the allocation is used without a NULL-check.
> The other allocations are checked for NULL before used.
>
> Is this correct?

It should NULL check, but I was mainly concerned with just using/matching the 
right allocator functions (HeapAlloc is failable just like CoTaskMemAlloc, 
and if there wasn't a NULL check before, there isn't now).

I can make in another patch to add missing NULL checks, though.



More information about the wine-devel mailing list