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
