diff options
author | Marc-André Lureau <marcandre.lureau@redhat.com> | 2024-11-15 16:18:13 +0400 |
---|---|---|
committer | Marc-André Lureau <marcandre.lureau@redhat.com> | 2025-02-03 21:16:07 +0400 |
commit | c0fb8e88cbbfc140800e614586b446193a3fd93a (patch) | |
tree | 8e947f6e5b4449cfb79f10fab584e94286ae84de | |
parent | 5b4109232e5c6db61fb02aa86b30170d93234acb (diff) | |
download | qemu-c0fb8e88cbbfc140800e614586b446193a3fd93a.zip qemu-c0fb8e88cbbfc140800e614586b446193a3fd93a.tar.gz qemu-c0fb8e88cbbfc140800e614586b446193a3fd93a.tar.bz2 |
plugins: fix -Werror=maybe-uninitialized false-positive
../contrib/plugins/cache.c:638:9: error: ‘l2_cache’ may be used uninitialized [-Werror=maybe-uninitialized]
638 | append_stats_line(rep, l1_dmem_accesses, l1_dmisses,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Is a false-positive, since cores > 1, so the variable is set in the
above loop.
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
-rw-r--r-- | contrib/plugins/cache.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/plugins/cache.c b/contrib/plugins/cache.c index 7baff86..7cfd3df 100644 --- a/contrib/plugins/cache.c +++ b/contrib/plugins/cache.c @@ -603,7 +603,7 @@ static int l2_cmp(gconstpointer a, gconstpointer b) static void log_stats(void) { int i; - Cache *icache, *dcache, *l2_cache; + Cache *icache, *dcache, *l2_cache = NULL; g_autoptr(GString) rep = g_string_new("core #, data accesses, data misses," " dmiss rate, insn accesses," |