On 03/14/2010 11:03 AM, Andrew Nguyen wrote:
> ---
> dlls/dxdiagn/container.c | 15 +++++------
> dlls/dxdiagn/tests/container.c | 49
> ++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 56 insertions(+), 8 deletions(-)
>
> -
> - TRACE("(%p, %u, %s, %u)\n", iface, dwIndex, debugstr_w(pwszContainer),
> cchContainer);
>
> - if (NULL == pwszContainer) {
> + TRACE("(%p, %u, %p, %u)\n", iface, dwIndex, pwszContainer, cchContainer);
> +
> + if (NULL == pwszContainer || 0 == cchContainer) {
> return E_INVALIDARG;
> }
Please add printing of debugstr_w(pwszContainer) back, after sanity checks.
It's something really useful to know.
