On 02/22/2010 01:57 AM, Piotr Caban wrote:
+ TRACE("(%p,%d,%p)\n", iface, index, pFuncDesc);
+ TRACE("{%d,%p,%p,%d,%d,%d,%d,%d,%d,%d,{%d},%d}\n", pFuncDesc->memid,
+ pFuncDesc->lprgscode, pFuncDesc->lprgelemdescParam,
pFuncDesc->funckind,
+ pFuncDesc->invkind, pFuncDesc->callconv, pFuncDesc->cParams,
+ pFuncDesc->cParamsOpt, pFuncDesc->oVft, pFuncDesc->cScodes,
+ pFuncDesc->elemdescFunc.tdesc.vt, pFuncDesc->wFuncFlags);
+
+ if(!pFuncDesc || (pFuncDesc->memid>0x7fffffff&&
pFuncDesc->memid!=MEMBERID_NIL))
+ return E_INVALIDARG;
Hi Piotr,
Your checking pFuncDesc for NULL after using it's members in the
TRACE(). At least Coverity will throw up an error on that.
--
Cheers,
Paul.
