From 067af6b9098858e786e71ef20cb91ad75c6e4ba4 Mon Sep 17 00:00:00 2001 From: Fabian Groffen Date: Tue, 7 Feb 2023 09:24:21 +0100 Subject: set: try to silence Coverity Now add_set_value can allocate a new set, Coverity thinks this will actually happen, despite in these cases the input set not being NULL. Help Coverity by adding a redundant if. Signed-off-by: Fabian Groffen --- libq/tree.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'libq') diff --git a/libq/tree.c b/libq/tree.c index a05a86e..f308c8d 100644 --- a/libq/tree.c +++ b/libq/tree.c @@ -1709,7 +1709,8 @@ tree_match_atom_cache_populate_cb(tree_pkg_ctx *ctx, void *priv) cat_ctx = get_set(atom->CATEGORY, cache); if (cat_ctx == NULL) { cat_ctx = tree_open_cat(tctx, "."); - cache = add_set_value(atom->CATEGORY, cat_ctx, NULL, cache); + if (cache != NULL) /* for static code analysers */ + add_set_value(atom->CATEGORY, cat_ctx, NULL, cache); /* get a pointer from the set */ cat_ctx->name = contains_set(atom->CATEGORY, cache); } -- cgit v1.2.3-65-gdbad