[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] hmp-cmd: fix a missing_break warning
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH] hmp-cmd: fix a missing_break warning |
Date: |
Tue, 24 Mar 2020 17:40:32 +0000 |
User-agent: |
Mutt/1.13.3 (2020-01-12) |
* Pan Nengyuan (address@hidden) wrote:
> Correcting zhang hailiang's email.
Queued
> On 3/18/2020 3:16 PM, Pan Nengyuan wrote:
> > This fix coverity issues 94417686:
> > 1260 break;
> > CID 94417686: (MISSING_BREAK)
> > 1261. unterminated_case: The case for value
> > "MIGRATION_PARAMETER_THROTTLE_TRIGGER_THRESHOLD" is not terminated by a
> > 'break' statement.
> > 1261 case MIGRATION_PARAMETER_THROTTLE_TRIGGER_THRESHOLD:
> > 1262 p->has_throttle_trigger_threshold = true;
> > 1263 visit_type_int(v, param, &p->throttle_trigger_threshold,
> > &err);
> > 1264 case MIGRATION_PARAMETER_CPU_THROTTLE_INITIAL:
> >
> > Fixes: dc14a470763c96fd9d360e1028ce38e8c3613a77
> > Reported-by: Euler Robot <address@hidden>
> > Signed-off-by: Pan Nengyuan <address@hidden>
> > ---
> > Cc: address@hidden
> > ---
> > monitor/hmp-cmds.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/monitor/hmp-cmds.c b/monitor/hmp-cmds.c
> > index 58724031ea..c882c9f3cc 100644
> > --- a/monitor/hmp-cmds.c
> > +++ b/monitor/hmp-cmds.c
> > @@ -1261,6 +1261,7 @@ void hmp_migrate_set_parameter(Monitor *mon, const
> > QDict *qdict)
> > case MIGRATION_PARAMETER_THROTTLE_TRIGGER_THRESHOLD:
> > p->has_throttle_trigger_threshold = true;
> > visit_type_int(v, param, &p->throttle_trigger_threshold, &err);
> > + break;
> > case MIGRATION_PARAMETER_CPU_THROTTLE_INITIAL:
> > p->has_cpu_throttle_initial = true;
> > visit_type_int(v, param, &p->cpu_throttle_initial, &err);
> >
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK