[FFmpeg-cvslog] avutil/tests/opt: Check av_set_options_string() for failure
Michael Niedermayer
git at videolan.org
Thu Jun 13 23:41:23 EEST 2024
ffmpeg | branch: release/5.1 | Michael Niedermayer <michael at niedermayer.cc> | Mon May 20 22:52:38 2024 +0200| [b2864f699e96f201c53ffe19165fd5be1348813e] | committer: Michael Niedermayer
avutil/tests/opt: Check av_set_options_string() for failure
This is test code after all so it should test things
Fixes: CID1518990 Unchecked return value
Sponsored-by: Sovereign Tech Fund
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
(cherry picked from commit e3481730ed9b3b781a0d85370826bcc57d601958)
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=b2864f699e96f201c53ffe19165fd5be1348813e
---
libavutil/tests/opt.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/libavutil/tests/opt.c b/libavutil/tests/opt.c
index 5799e45c6a..2cc41647f1 100644
--- a/libavutil/tests/opt.c
+++ b/libavutil/tests/opt.c
@@ -217,6 +217,7 @@ int main(void)
{
TestContext test_ctx = { 0 };
char *buf;
+ int ret;
test_ctx.class = &test_class;
av_log_set_level(AV_LOG_QUIET);
@@ -227,8 +228,10 @@ int main(void)
av_opt_free(&test_ctx);
memset(&test_ctx, 0, sizeof(test_ctx));
test_ctx.class = &test_class;
- av_set_options_string(&test_ctx, buf, "=", ",");
+ ret = av_set_options_string(&test_ctx, buf, "=", ",");
av_free(buf);
+ if (ret < 0)
+ printf("Error ret '%d'\n", ret);
if (av_opt_serialize(&test_ctx, 0, 0, &buf, '=', ',') >= 0) {
printf("%s\n", buf);
av_free(buf);
More information about the ffmpeg-cvslog
mailing list