Re: [FFmpeg-devel] [PATCH 1/7] avcodec/ffv1dec: add missing error messages when a frame is invalid

2018-03-09 Thread Michael Niedermayer
On Thu, Mar 08, 2018 at 02:48:19AM +0100, Jerome Martinez wrote: > On 08/03/2018 01:17, Michael Niedermayer wrote: > > > >In the cases where the error is about a scalar value, that value should > >be printed in the error message (unless it was alread printed elsewhere) > > Patch modified, showing

Re: [FFmpeg-devel] [PATCH 1/7] avcodec/ffv1dec: add missing error messages when a frame is invalid

2018-03-07 Thread Jerome Martinez
On 08/03/2018 01:17, Michael Niedermayer wrote: In the cases where the error is about a scalar value, that value should be printed in the error message (unless it was alread printed elsewhere) Patch modified, showing the scalar value. From 13db9fc4da1d0e531e516bd87d084233e231f0e7 Mon Sep 17 00

Re: [FFmpeg-devel] [PATCH 1/7] avcodec/ffv1dec: add missing error messages when a frame is invalid

2018-03-07 Thread Michael Niedermayer
On Wed, Mar 07, 2018 at 04:45:20PM +0100, Jerome Martinez wrote: > A buggy file (before the patch preventing such issue is applied): > ffmpeg -y -f lavfi -i mandelbrot=s=31x31 -vframes 1 -c ffv1 -slices 961 > a.mkv > > Then with: > ffmpeg -y -f lavfi -i mandelbrot=s=31x31 -vframes 1 source.jpg > f

[FFmpeg-devel] [PATCH 1/7] avcodec/ffv1dec: add missing error messages when a frame is invalid

2018-03-07 Thread Jerome Martinez
A buggy file (before the patch preventing such issue is applied): ffmpeg -y -f lavfi -i mandelbrot=s=31x31 -vframes 1 -c ffv1 -slices 961 a.mkv Then with: ffmpeg -y -f lavfi -i mandelbrot=s=31x31 -vframes 1 source.jpg ffmpeg -y -i a.mkv a.jpg There is no error message despite the fact the stre