[FFmpeg-devel] [PATCH 1/2] lavf/dvenc: improve error messaging

Stefano Sabatini stefasab at gmail.com
Wed Jan 24 01:48:23 EET 2024


On date Sunday 2024-01-21 20:18:28 +0100, Stefano Sabatini wrote:
> On date Sunday 2024-01-21 18:39:19 +0100, Anton Khirnov wrote:
> > Quoting Stefano Sabatini (2024-01-21 11:30:27)
> [...]
> > Also, can the second case even trigger? Seems like the block above
> > ensures n_ast is never larger than 2.
> 
> Yes, this seems a miss from commit
> eafa8e859297813dcf11110e6b43e85720be0a5f.

Updated, keeping the somehow faulty logic as unrelated to the change
(should be fixed separately).

Adding Baptiste in CC: for the issue with the number of channels.

The obvious fix seems to restore the limit of max 3 channels, and
delete ast[2] and ast[3], but maybe Baptiste can hint at the proper
fix.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavf-dvenc-improve-error-messaging.patch
Type: text/x-diff
Size: 7724 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240124/9052d726/attachment.patch>


More information about the ffmpeg-devel mailing list