Re: [FFmpeg-devel] [PATCH 2/2] avcodec/dvdsubdec: reject some broken packets

2015-09-22 Thread wm4
On Mon, 21 Sep 2015 23:33:16 +0200 Michael Niedermayer wrote: > On Mon, Sep 21, 2015 at 06:25:31PM +0200, wm4 wrote: > > If cmd_pos is broken, this would just keep accumulating packets in the > > reassembly buffer, until it fails and flushes the buffer on overflow. > > Since

[FFmpeg-devel] [PATCH 2/2] avcodec/dvdsubdec: reject some broken packets

2015-09-21 Thread wm4
If cmd_pos is broken, this would just keep accumulating packets in the reassembly buffer, until it fails and flushes the buffer on overflow. Since packets are usually rather small, this will take a lot of subtitle packets. The perceived effect is that subtitles are not displayed anymore after the

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/dvdsubdec: reject some broken packets

2015-09-21 Thread Michael Niedermayer
On Mon, Sep 21, 2015 at 06:25:31PM +0200, wm4 wrote: > If cmd_pos is broken, this would just keep accumulating packets in the > reassembly buffer, until it fails and flushes the buffer on overflow. > Since packets are usually rather small, this will take a lot of subtitle > packets. The perceived