Re: [FFmpeg-devel] [PATCH]lavc/mjpegdec: Only set SAR if resolution is available

2016-01-27 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > > The attached patch also fixes ticket #4479 for me. > also ok, maybe add a <= 0 check for the 2 parameters Applied with that change. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmp

Re: [FFmpeg-devel] [PATCH]lavc/mjpegdec: Only set SAR if resolution is available

2016-01-19 Thread Michael Niedermayer
On Tue, Jan 19, 2016 at 09:16:26AM +0100, Carl Eugen Hoyos wrote: > On Tuesday 19 January 2016 02:09:06 am Michael Niedermayer wrote: > > On Tue, Jan 19, 2016 at 12:33:40AM +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #4479 here. > > > > > > Please comment, Carl

Re: [FFmpeg-devel] [PATCH]lavc/mjpegdec: Only set SAR if resolution is available

2016-01-19 Thread Carl Eugen Hoyos
On Tuesday 19 January 2016 02:09:06 am Michael Niedermayer wrote: > On Tue, Jan 19, 2016 at 12:33:40AM +0100, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes ticket #4479 here. > > > > Please comment, Carl Eugen > > if w/h is set at that point then their values may be unrelated to the

Re: [FFmpeg-devel] [PATCH]lavc/mjpegdec: Only set SAR if resolution is available

2016-01-18 Thread Michael Niedermayer
On Tue, Jan 19, 2016 at 12:33:40AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4479 here. > > Please comment, Carl Eugen if w/h is set at that point then their values may be unrelated to the actual w/h values thus the set_sar there is wrong even if w/h is set the patch

[FFmpeg-devel] [PATCH]lavc/mjpegdec: Only set SAR if resolution is available

2016-01-18 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4479 here. Please comment, Carl Eugen diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index b1c5b67..0e95ebd 100644 --- a/libavcodec/mjpegdec.c +++ b/libavcodec/mjpegdec.c @@ -1653,7 +1653,8 @@ static int mjpeg_decode_app(MJpegDecodeContext *s)