2016-12-10 16:11 GMT+01:00 Andreas Cadhalpun <andreas.cadhal...@googlemail.com>:
> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com>
> ---
> libavutil/opt.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/libavutil/opt.c b/libavutil/opt.c
> index f855ccb..f713d3f 100644
> --- a/libavutil/opt.c
> +++ b/libavutil/opt.c
> @@ -32,6 +32,7 @@
> #include "common.h"
> #include "dict.h"
> #include "eval.h"
> +#include "imgutils.h"
> #include "log.h"
> #include "parseutils.h"
> #include "pixdesc.h"
> @@ -325,8 +326,15 @@ static int set_string_image_size(void *obj, const
> AVOption *o, const char *val,
> return 0;
> }
> ret = av_parse_video_size(dst, dst + 1, val);
> - if (ret < 0)
> + if (ret < 0) {
> av_log(obj, AV_LOG_ERROR, "Unable to parse option value \"%s\" as
> image size\n", val);
> + return ret;
> + }
> + ret = av_image_check_size(*dst, *(dst + 1), 0, obj);
> + if (ret < 0) {
> + *dst = 0;
> + *(dst + 1) = 0;
> + }
Doesn't this break valid usecases?
Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
