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.