On Wed, Mar 08, 2017 at 01:40:10PM +0100, wm4 wrote: > Apparently the demuxer outputs the wrong padding for HE-AAC (based on > the raw sample rate, or so). aacdec contains a hack to adjust the muxer > padding accordingly before it's used to trim the decoder output. This > modified the packet side data, which in combination with the old > decoding API would change the packet the user passed to the decoder. > This is clearly not allowed, and it breaks running some gapless fate > tests with "-fflags +keepside" applied (without keepside, the packet > metadata is typically newly allocated, essentially making a copy and not > modifying the user's input packet). > > This should probably be fixed in the demuxer (and consequently also the > muxer), but for now only fix the immediate problem. > > Regression since 946ed78f5f8 (2012). > --- > libavcodec/aacdec_template.c | 8 ++------ > libavcodec/internal.h | 2 ++ > libavcodec/utils.c | 4 +++- > 3 files changed, 7 insertions(+), 7 deletions(-)
this patch LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many things microsoft did are stupid, but not doing something just because microsoft did it is even more stupid. If everything ms did were stupid they would be bankrupt already.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
