howto fix bug 13462

[Louis Lenders]

Rob Shearman <robertshearman <at> gmail.com> writes:
Thanks very much for your comments Rob,
I've updated the patch here:
http://bugs.winehq.org/attachment.cgi?id=16397

Hopefully correct now?

But now got another problem with the test, i changed it a bit like here:
http://bugs.winehq.org/attachment.cgi?id=16430

This test passes fine in Xp, but not in wine, more specific, this part fails:
pCoInitializeEx(NULL, COINIT_APARTMENTTHREADED);
.
.
.
.     if(hr == S_OK) 
        refs = IContextCallback_Release(pComThreadingInfo);
    
    ok(refs == 0, "pComThreadingInfo should have 0 refs instead of %d refs\n",
refs);
 
In windows it shows the refcount should be zero in the end, but with my patch
and this test, it doesn't (refcount =2 in the end). I added several traces to
the code to show how the refcount was in/decreased, but wasn't able to figure
out what goes wrong, it's quite mysterious. Whereas this test fails, the one
that starts with 
pCoInitializeEx(NULL, COINIT_MULTITHREADED) passes just fine in wine. Am i still
missing something?