On Wed, Sep 30, 2015 at 02:54:11PM +0200, wm4 wrote: > Commits 43bc5cf9 and c5371f77 add code for skipping initial zeros in mp3 > packets. (There are no explanations, no samples.) > > This code forgot to report to the user that data was skipped at all. IT > should do this by adding the skipped amount of bytes to the amount of > consumed bytes returned by the decode call. > > Since audio codecs allow partial packet decoding, the user application > has to rely on the return value. It will remove the data reported as > consumed by the decoder, and feed it to the decoder again. This resulted > in the mp3 frame after the zero region to be decoded over and over > again, until the zero region was finally skipped by the application. > > Fixes trac ticket #4890. > --- > libavcodec/mpegaudiodec_template.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-)
LGTM thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel