[FFmpeg-devel] [PATCH v3 3/4] lavfi: check the validity of formats lists.

[Nicolas George]

Andreas Rheinhardt (12020-08-24):
> >>> +    if (fmts->all_layouts < fmts->all_counts ||
> >>> +        (!fmts->all_layouts && !fmts->nb_channel_layouts)) {

> The above check already contains "|| (!fmts->all_layouts &&
> !fmts->nb_channel_layouts)". So if everything is zero, you get the error
> for an inconsistent generic list.

Oh, right, I had considered doing both tests at once, but then separated
them for a different message, and forgot. I locally removed the second
line above.

Sorry.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200824/7b18e7ca/attachment.sig>