[FFmpeg-devel] [PATCH] avcodec/ffv1dec: fix crash if number of slice counts change midstream

[Michael Niedermayer]

On Fri, Sep 25, 2015 at 12:29:04AM +0200, Paul B Mahol wrote:
> On 9/24/15, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Thu, Sep 24, 2015 at 09:45:35PM +0200, Paul B Mahol wrote:
> >> Everyting points this is currently unsupported.
> >>
> >> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> >> ---
> >>  libavcodec/ffv1dec.c | 15 +++++++++++----
> >>  1 file changed, 11 insertions(+), 4 deletions(-)
> >
> > fixed differently (so slice_counts can change, not really tested
> > though due to lack of encoder support)
> 
> That fix decodes nonsense for 2nd part when concatinating 2 nuts when
> 1st have 16
> slices and 2nd have 4 slices.
> 
> Just create 2 nuts with different slice number and concat them with concat
> demuxer and copy codec.

why would this be a valid file ?

theres a global horizontal and vertical slice count and on this
raster slices can occupy some rectangular area, currently only 1x1
is supported. Theres no support to change the raster resolution
itself midstream. 

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150925/2d81af82/attachment.sig>