Re: [PATCH 0/2] MR7465: winegstreamer: Validate the value of cbSize in aac decoder.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/aac_decoder.c:
>  if (!count)
>      return MF_E_INVALIDMEDIATYPE;
>
> - if (wfx.Format.nChannels >= ARRAY_SIZE(default_channel_mask) || !wfx.Format.nSamplesPerSec || !wfx.Format.cbSize)
> - if (wfx.Format.nChannels >= ARRAY_SIZE(default_channel_mask) || !wfx.Format.nSamplesPerSec
> -
>        /* 2 is the minimum size of AudioSpecificConfig() */
> -
>        || wfx.Format.cbSize < 2 + CBSIZE(WAVEFORMATEXTENSIBLE) || (IsEqualGUID(&wfx.SubFormat, &MFAudioFormat_AAC)
> -
>        && wfx.Format.cbSize < 2 + CBSIZE(WAVEFORMATEXTENSIBLE) + CBSIZE(HEAACWAVEINFO)))
>    return MF_E_INVALIDMEDIATYPE;

I'm not sure it's worth adding more checks here, it was mostly just to pass the tests but I don't think anything should care about it failing with specific sizes. If some tests are getting in the way they can be changed to todo_wine / removed.