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

Paul B Mahol onemda at gmail.com
Fri Sep 25 08:27:29 CEST 2015


Dana 25. 9. 2015. 01:10 osoba "Michael Niedermayer" <michaelni at gmx.at>
napisala je:
>
> 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.

But it doesn't complain at all, so you get invalid output.

> [...]
>
> --
> 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
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list