[FFmpeg-cvslog] avopts: Fix random values being printed in error messages:

Michael Niedermayer git at videolan.org
Sun Nov 20 15:16:50 CET 2011


ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Sun Nov 20 14:44:41 2011 +0100| [a9011623ea95f051f740bffd2d473f28fd1f250e] | committer: Michael Niedermayer

avopts: Fix random values being printed in error messages:
Bug introduced in:
commit 7bb1807c2dd9623842f1c6d454e7f2892b5bc8d7
Author: Anton Khirnov <anton at khirnov.net>
Date:   Tue Aug 23 12:58:49 2011 +0200

    AVOptions: refactor set_number/write_number

    write_number() does the actual writing of the supplied
    number to destination. Move finding the option and choosing destination
    address out of it.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=a9011623ea95f051f740bffd2d473f28fd1f250e
---

 libavutil/opt.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/libavutil/opt.c b/libavutil/opt.c
index ca31be3..1d7ef38 100644
--- a/libavutil/opt.c
+++ b/libavutil/opt.c
@@ -80,7 +80,7 @@ static int read_number(const AVOption *o, void *dst, double *num, int *den, int6
 static int write_number(void *obj, const AVOption *o, void *dst, double num, int den, int64_t intnum)
 {
     if (o->max*den < num*intnum || o->min*den > num*intnum) {
-        av_log(obj, AV_LOG_ERROR, "Value %lf for parameter '%s' out of range\n", num, o->name);
+        av_log(obj, AV_LOG_ERROR, "Value %f for parameter '%s' out of range\n", num*intnum/den, o->name);
         return AVERROR(ERANGE);
     }
 



More information about the ffmpeg-cvslog mailing list