On 4/21/2010 13:14, Alistair Leslie-Hughes wrote: > + ret = IUnknown_QueryInterface(pUnk,&IID_IUnknown, (void**)&pUnkRemote); > There's no point to do it. You already have IUnknown pointer. It's better to remove test for that case I think.