[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] monitor: fix maybe-uninitialized
From: |
Markus Armbruster |
Subject: |
Re: [PATCH] monitor: fix maybe-uninitialized |
Date: |
Wed, 11 Dec 2019 17:10:17 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> From: Marc-André Lureau <address@hidden>
>
> ../monitor/misc.c: In function ‘mon_get_cpu_sync’:
> /home/elmarco/src/qq/include/sysemu/hw_accel.h:22:9: error: ‘cpu’ may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> 22 | kvm_cpu_synchronize_state(cpu);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> ../monitor/misc.c:397:15: note: ‘cpu’ was declared here
> 397 | CPUState *cpu;
> | ^~~
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> monitor/misc.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/monitor/misc.c b/monitor/misc.c
> index 3baa15f..a74cff3 100644
> --- a/monitor/misc.c
> +++ b/monitor/misc.c
> @@ -394,7 +394,7 @@ int monitor_set_cpu(int cpu_index)
> /* Callers must hold BQL. */
> static CPUState *mon_get_cpu_sync(bool synchronize)
> {
> - CPUState *cpu;
> + CPUState *cpu = NULL;
>
> if (cur_mon->mon_cpu_path) {
> cpu = (CPUState *) object_resolve_path_type(cur_mon->mon_cpu_path,
> @@ -411,6 +411,7 @@ static CPUState *mon_get_cpu_sync(bool synchronize)
> monitor_set_cpu(first_cpu->cpu_index);
> cpu = first_cpu;
> }
> + assert(cpu != NULL);
> if (synchronize) {
> cpu_synchronize_state(cpu);
> }
Reviewed-by: Markus Armbruster <address@hidden>