Commit 6d72540d42 for ffmpeg
commit 6d72540d429b3c6aad77df4f5382eaa7330d8862
Author: zhang xingxing <2388969553@qq.com>
Date: Mon Sep 21 06:13:29 2026 +0200
avutil/eval: evaluate the first argument of clip() only once
Found-by: zhang xingxing
Fixes: double evaluation of the clip() argument
Fixes: QI0lR8gkbkbN
Fixes: swarm-e_clip-double-evaluates-first-parameter-8c8411
diff --git a/libavutil/eval.c b/libavutil/eval.c
index 2bd07b98c1..9b5ad21849 100644
--- a/libavutil/eval.c
+++ b/libavutil/eval.c
@@ -222,7 +222,7 @@ static double eval_expr(Parser *p, AVExpr *e)
double min = eval_expr(p, e->param[1]), max = eval_expr(p, e->param[2]);
if (isnan(min) || isnan(max) || isnan(x) || min > max)
return NAN;
- return e->value * av_clipd(eval_expr(p, e->param[0]), min, max);
+ return e->value * av_clipd(x, min, max);
}
case e_between: {
double d = eval_expr(p, e->param[0]);