[FFmpeg-devel] [PATCH 0/6] avformat/movenc: normalize on AC-3 parser usage

Jan Ekström jeebjp at gmail.com
Tue Jun 7 09:58:46 EEST 2022


On Fri, Jun 3, 2022 at 3:46 PM Jan Ekström <jeebjp at gmail.com> wrote:
>
> The simplified parsing currently in `mov_write_ac3_tag` trusts the content
> of the packets a bit too much (the AC-3 parser returns all data fed to it,
> including any possible data before the start code), while the existing E-AC-3
> logic does proper header validation by utilizing the (E-)AC-3 parser.
>
> Thus, normalize on AC-3 parser usage for both AC-3 and E-AC-3.

Ping on this patch set, as this seems to improve the state of affairs
with the input being MPEG-TS muxes where PES packets do not start with
a start code (the previous ad-hoc parsing was way too trusting and
would write bogus values based on the last N bytes of a previous
packet into the AC-3 sample description).

When applying, `avcodec/ac3_parser{,_internal}: expose AC-3
bit_rate_code` will have a minor bump as technically a
private-yet-utilized-from-lavf structure has something appended to it.

Jan


More information about the ffmpeg-devel mailing list