[FFmpeg-devel] [PATCH 2/2] avfilter: handle duplicates in the options string
Anton Khirnov
anton at khirnov.net
Tue Mar 15 10:58:12 EET 2022
Use the same logic as fftools/cmdutils - when a flag-type option starts
with [+-], append it to the existing value.
---
libavfilter/avfilter.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c
index 030f1823ee..a6f2181efb 100644
--- a/libavfilter/avfilter.c
+++ b/libavfilter/avfilter.c
@@ -860,13 +860,17 @@ static int process_options(AVFilterContext *ctx, AVDictionary **options,
return ret;
}
} else {
- av_dict_set(options, key, value, 0);
- if (!av_opt_find(ctx->priv, key, NULL, 0, AV_OPT_SEARCH_CHILDREN | AV_OPT_SEARCH_FAKE_OBJ)) {
+ o = av_opt_find(ctx->priv, key, NULL, 0,
+ AV_OPT_SEARCH_CHILDREN | AV_OPT_SEARCH_FAKE_OBJ);
+ if (!o) {
av_log(ctx, AV_LOG_ERROR, "Option '%s' not found\n", key);
av_free(value);
av_free(parsed_key);
return AVERROR_OPTION_NOT_FOUND;
}
+ av_dict_set(options, key, value,
+ (o->type == AV_OPT_TYPE_FLAGS &&
+ (value[0] == '-' || value[0] == '+')) ? AV_DICT_APPEND : 0);
}
av_free(value);
--
2.34.1
More information about the ffmpeg-devel
mailing list