[FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: configure min_hard_comp unconditionally
Zhao Zhili
quinkblack at foxmail.com
Sat Sep 17 07:30:01 EEST 2022
From: Zhao Zhili <zhilizhao at tencent.com>
There are two issues here. Firstly, the floating-point comparison
is always true. Seconly, the code depends on the default value of
min_hard_comp implicitly, which can be dangerous.
Partially fixes ticket 9859.
---
fftools/ffmpeg_filter.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c
index 7a5308425d..17928cea2c 100644
--- a/fftools/ffmpeg_filter.c
+++ b/fftools/ffmpeg_filter.c
@@ -896,8 +896,7 @@ static int configure_input_audio_filter(FilterGraph *fg, InputFilter *ifilter,
char args[256] = {0};
av_strlcatf(args, sizeof(args), "async=%d", audio_sync_method);
- if (audio_drift_threshold != 0.1)
- av_strlcatf(args, sizeof(args), ":min_hard_comp=%f", audio_drift_threshold);
+ av_strlcatf(args, sizeof(args), ":min_hard_comp=%f", audio_drift_threshold);
if (!fg->reconfiguration)
av_strlcatf(args, sizeof(args), ":first_pts=0");
AUTO_INSERT_FILTER_INPUT("-async", "aresample", args);
--
2.35.3
More information about the ffmpeg-devel
mailing list