[FFmpeg-devel] [PATCH 5/8] ffmdec: break infinite resync loop

[Michael Niedermayer]

On Mon, Mar 09, 2015 at 03:14:50PM +0100, Andreas Cadhalpun wrote:
> On 09.03.2015 13:59, Michael Niedermayer wrote:
> >i was thinking more about limiting the backward seek before resync
> >to the last resync position +1 if there was a previous resync
> >so that resync which moves forward could not end before. This would
> >avoid the failure and allow the demuxer to continue, or at least thats
> >the idea
> 
> I see, new patch attached.
> The +1 is not necessary, because it always reads the PACKET_ID field
> before resyncing.
> 
> Best regards,
> Andreas
> 

>  ffmdec.c |    9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 2f242fcf689ae8307a818b8a219d08af7c27921e  0001-ffmdec-limit-the-backward-seek-to-the-last-resync-po.patch
> From 6bcd1b4a906e2da39f6361db19826a337ac5a745 Mon Sep 17 00:00:00 2001
> From: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
> Date: Mon, 9 Mar 2015 14:59:44 +0100
> Subject: [PATCH] ffmdec: limit the backward seek to the last resync position
> 
> If resyncing leads to the same position as previously, it will again
> lead to a resync attempt, resulting in an infinite loop.
> 
> Thus don't seek back beyond the last syncpoint.

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Asymptotically faster algorithms should always be preferred if you have
asymptotical amounts of data
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150309/97ea789c/attachment.asc>