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