Re: inetcomm: possible null pointer dereferences

[Hans Leidekker]

On Thursday 04 December 2008 21:32:37 Gregor Schneider wrote:
> - TRACE("(%s)\n", pszEmailFrom);
> - if (!pszEmailFrom)
>       return E_INVALIDARG;
> - TRACE("(%s)\n", pszEmailFrom);
> -
> - len = sizeof(szCommandFormat) - 2 /* "%s" */ + strlen(pszEmailFrom);
> -

Instead of moving the trace down you should use debugstr_a(), which handles
the NULL pointer for you.

-Hans