diff options
| author | Peter Maydell <peter.maydell@linaro.org> | 2022-07-19 14:48:53 +0100 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2022-07-22 19:01:44 +0200 |
| commit | d12dd9c7ee0ecab96efc21a263772fef2fff3ac6 (patch) | |
| tree | f81a8252fc68860a6a321b7c5f15eb196896dea9 /accel/kvm/kvm-all.c | |
| parent | b67de91e0d4cae883a038f31b2207a1aaef5e51c (diff) | |
| download | qemu-d12dd9c7ee0ecab96efc21a263772fef2fff3ac6.zip qemu-d12dd9c7ee0ecab96efc21a263772fef2fff3ac6.tar.gz qemu-d12dd9c7ee0ecab96efc21a263772fef2fff3ac6.tar.bz2 | |
accel/kvm: Avoid Coverity warning in query_stats()
Coverity complains that there is a codepath in the query_stats()
function where it can leak the memory pointed to by stats_list. This
can only happen if the caller passes something other than
STATS_TARGET_VM or STATS_TARGET_VCPU as the 'target', which no
callsite does. Enforce this assumption using g_assert_not_reached(),
so that if we have a future bug we hit the assert rather than
silently leaking memory.
Resolves: Coverity CID 1490140
Fixes: cc01a3f4cadd91e6 ("kvm: Support for querying fd-based stats")
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20220719134853.327059-1-peter.maydell@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'accel/kvm/kvm-all.c')
| -rw-r--r-- | accel/kvm/kvm-all.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c index 99aede7..f165074 100644 --- a/accel/kvm/kvm-all.c +++ b/accel/kvm/kvm-all.c @@ -4014,7 +4014,7 @@ static void query_stats(StatsResultList **result, StatsTarget target, stats_list); break; default: - break; + g_assert_not_reached(); } } |