On 16.08.2006 04:23, Benjamin Arai wrote:
> if(pbstrHelpString)
> + {
> *pbstrHelpString=SysAllocString(This->DocString);
> + if(!*pbstrHelpString)
> + return STG_E_INSUFFICIENTMEMORY;
> + }
> if(pdwHelpStringContext)
> *pdwHelpStringContext=This->dwHelpContext;
> if(pbstrHelpStringDll)
> + {
> *pbstrHelpStringDll=SysAllocString(This->HelpStringDll);
> + if(!pbstrHelpStringDll)
> + return STG_E_INSUFFICIENTMEMORY;
> + }
Hm, shouldn't pbstrHelpString be freed in case the 2nd allocation fails?
-f.r.
