Re: [PATCH] amstream: Check params in IAMMultiMediaStreamImpl_OpenFile.

[Nikolay Sivov]

On 10/12/2012 00:59, Christian Costa wrote:
> ---
>   dlls/amstream/amstream.c |   12 +++++++++---
>   1 file changed, 9 insertions(+), 3 deletions(-)

> -    TRACE("(%p/%p)->(%s,%x)\n", This, iface, debugstr_w(pszFileName), dwFlags);
> +    TRACE("(%p/%p)->(%s,%x)\n", This, iface, debugstr_w(filename), flags);
> +
> +    if (!filename)
> +        return E_POINTER;
> +
> +    if (flags & ~(AMMSF_NOCLOCK | AMMSF_NORENDER | AMMSF_RENDERALLSTREAMS | AMMSF_RENDERTOEXISTING | AMMSF_RUN))
> +        return E_INVALIDARG;

This needs tests.