icmstream mem leak fix

Mike McCormack mike at codeweavers.com
Tue Nov 30 02:12:23 CST 2004

James Hawkins wrote:

> I just want to make sure I've got this.  Do we want to release even if
> the call to QueryInterface suceeds?

The old code sets ref=0, then just free's the memory.  My method would 
set ref=1, then call Release() which decrements ref and free's memory.

If QueryInterface succeeds, then it will increment the reference count 
again, so we'll only end up free'ing memory in Release if the 
QueryInterface fails.  It's a more COM-ish way of doing things... the 
original code still works though.


More information about the wine-devel mailing list