Hello Christian,
On 11/15/2012 09:43 AM, Christian Costa wrote:
> Fixes bug 32185.
> ---
> dlls/amstream/mediastreamfilter.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/dlls/amstream/mediastreamfilter.c
> b/dlls/amstream/mediastreamfilter.c
> index 9b2bd8e..d5388e5 100644
> --- a/dlls/amstream/mediastreamfilter.c
> +++ b/dlls/amstream/mediastreamfilter.c
> @@ -132,6 +132,8 @@ static HRESULT WINAPI BasePinImp_GetMediaType(BasePin
> *This, int index, AM_MEDIA
> MSPID purpose_id;
> int i;
>
> + ZeroMemory(amt, sizeof(*amt));
> +
this looks "odd". There is no check if amt is NULL, at least that's what
caught my attention. So I've looked around and stumbled upon the
AM_MEDIA_TYPE structure documentation
http://msdn.microsoft.com/en-us/library/windows/desktop/dd373477%28v=vs.85%29.aspx
"pUnk
Not used. Set to NULL."
Setting that to NULL in amstream's BasePinImp_GetMediaType() fixes the
crash for me. But who's responsibility is to set / check that? That
function or the caller aka test_media_streams() ?
bye
michael
