Bug#695593: Fwd: [libav-devel] Fwd: Bug#695593: libavcodec53: Transcoding wmav2 file leaves ffmpeg in infinite loop

[Laurent Bigonville]

Hi Justin,

I'm still interested to see this patch being merged.

I can confirm that this patch is fixing the infinite loop. I'm using it
for sometimes now without seeing any other issues.

Do you need more information?

Cheers

Laurent Bigonville


Le Mon, 20 May 2013 12:44:25 +0200,
Reinhard Tartler <siretart at tauware.de> a écrit :

> tags 695593 -patch +help
> severity 695593 minor
> stop
> 
> Laurent,
> 
> are you still intersted in seeing this patch in Libav? If so, please
> followup with Justin (CC'ed)
> 
> Regards,
> Reinhard
> 
> On Mo, Jan 14, 2013 at 14:39:01 (CET), Reinhard Tartler wrote:
> 
> > FYI,
> >
> > I guess the attached message did neither reach the bug nor you, so
> > I'm forwarding it.
> >
> >
> > ---------- Forwarded message ----------
> > From: Justin Ruggles <justin.ruggles at gmail.com>
> > Date: Mon, Dec 10, 2012 at 5:18 PM
> > Subject: Re: [libav-devel] Fwd: Bug#695593: libavcodec53:
> > Transcoding wmav2 file leaves ffmpeg in infinite loop
> > To: libav development <libav-devel at libav.org>
> >
> >
> > On 12/10/2012 10:44 AM, Reinhard Tartler wrote:
> >> Hi,
> >>
> >> could someone please look at the attached patch? From a quick look
> >> on gitweb, it looks like it would apply to current libav master.
> >> If it does, please add a CC: libav-stable @ to the commit message.
> >> Thanks!
> >>
> >> Reinhard
> >>
> >> ---------- Forwarded message ----------
> >> From: Laurent Bigonville <bigon at debian.org>
> >> Date: Mon, Dec 10, 2012 at 3:14 PM
> >> Subject: Bug#695593: libavcodec53: Transcoding wmav2 file leaves
> >> ffmpeg in infinite loop
> >> To: Debian Bug Tracking System <submit at bugs.debian.org>
> >>
> >>
> >> Package: libavcodec53
> >> Version: 6:0.8.4-1
> >> Severity: important
> >> Tags: patch
> >>
> >> Hi,
> >>
> >> In some situations, transcoding a wmav2 file can lead to a
> >> infinite loop (which is leaking a lot of memory).
> >>
> >> A patch has been merged in ffmpeg codebase, but this is still
> >> present in libav. (3cb64e327a9861f2c8fff0b3850fcd5d14acdbe3 on
> >> ffmpeg git repository).
> >>
> >> Please find attached the backport of the patch.
> >>
> >> Cheers
> >>
> >> Laurent Bigonville
> >>
> >> PS: Bug on the ffmpeg track:
> >> http://avcodec.org/trac/ffmpeg/ticket/286
> >>
> >> -- System Information:
> >> Debian Release: wheezy/sid
> >>   APT prefers unstable
> >>   APT policy: (500, 'unstable'), (1, 'experimental')
> >> Architecture: amd64 (x86_64)
> >> Foreign Architectures: i386
> >>
> >> Kernel: Linux 3.6-trunk-amd64 (SMP w/8 CPU cores)
> >> Locale: LANG=fr_BE.utf8, LC_CTYPE=fr_BE.utf8 (charmap=UTF-8)
> >> Shell: /bin/sh linked to /bin/dash
> >>
> >> Versions of packages libavcodec53 depends on:
> >> ii  libavutil51            6:0.8.4-1
> >> ii  libc6                  2.13-37
> >> ii  libdirac-encoder0      1.0.2-6
> >> ii  libgsm1                1.0.13-4
> >> ii  libmp3lame0            3.99.5+repack1-3
> >> ii  libopenjpeg2           1.3+dfsg-4.6
> >> ii  libschroedinger-1.0-0  1.0.11-2
> >> ii  libspeex1              1.2~rc1-7
> >> ii  libtheora0             1.1.1+dfsg.1-3.1
> >> ii  libva1                 1.0.15-4
> >> ii  libvorbis0a            1.3.2-1.3
> >> ii  libvorbisenc2          1.3.2-1.3
> >> ii  libvpx1                1.1.0-1
> >> ii  libx264-123            2:0.123.2189+git35cf912-1
> >> ii  libxvidcore4           2:1.3.2-9
> >> ii  multiarch-support      2.13-37
> >> ii  zlib1g                 1:1.2.7.dfsg-13
> >>
> >> libavcodec53 recommends no packages.
> >>
> >> libavcodec53 suggests no packages.
> >>
> >> -- no debconf information
> >>
> >> _______________________________________________
> >> pkg-multimedia-maintainers mailing list
> >> pkg-multimedia-maintainers at lists.alioth.debian.org
> >> http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
> >>
> >>
> >> -- regards, Reinhard
> >>
> >>
> >> fix_infiniteloop.patch
> >>
> >>
> >> --- a/libavcodec/wmadec.c
> >> +++ b/libavcodec/wmadec.c
> >> @@ -823,7 +823,8 @@ static int wma_decode_superframe(AVCodec
> >>                 buf_size, s->block_align);
> >>          return AVERROR_INVALIDDATA;
> >>      }
> >> -    buf_size = s->block_align;
> >> +    if(s->block_align)
> >> +        buf_size = s->block_align;
> >>
> >>      init_get_bits(&s->gb, buf, buf_size*8);
> >>
> >> @@ -921,7 +922,7 @@ static int wma_decode_superframe(AVCodec
> >>      *got_frame_ptr   = 1;
> >>      *(AVFrame *)data = s->frame;
> >>
> >> -    return s->block_align;
> >> +    return buf_size;
> >>   fail:
> >>      /* when error, we reset the bit reservoir */
> >>      s->last_superframe_len = 0;
> >
> > The patch looks ok, although it won't apply to HEAD.
> >
> > -Justin
> > _______________________________________________
> > libav-devel mailing list
> > libav-devel at libav.org
> > https://lists.libav.org/mailman/listinfo/libav-devel
>