On Fri, Jun 3, 2022 at 3:46 PM Jan Ekström <jee...@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 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".