On Wed, 7 Mar 2018 14:01:27 +0000 "Dr. David Alan Gilbert" <[email protected]> wrote:
> * Igor Mammedov ([email protected]) wrote: > > Ban it for now, if someone would need it to work early, > > one would have to implement checks if HMP command is valid > > at preconfig state. > > > > Signed-off-by: Igor Mammedov <[email protected]> > > --- > > --- > > monitor.c | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/monitor.c b/monitor.c > > index f499250..fcb5386 100644 > > --- a/monitor.c > > +++ b/monitor.c > > @@ -3097,6 +3097,10 @@ static void handle_hmp_command(Monitor *mon, const > > char *cmdline) > > > > trace_handle_hmp_command(mon, cmdline); > > > > + if (runstate_check(RUN_STATE_PRECONFIG)) { > > + monitor_printf(mon, "HMP not available in precofig state\n"); > > + } > > But you've not returned, so the command will still get parsed? yep, it's bug. I've already fixed in v4. Plan to post simplified v4 this Friday if nothing goes south. > > Dave > > > cmd = monitor_parse_command(mon, cmdline, &cmdline, mon->cmd_table); > > if (!cmd) { > > return; > > -- > > 2.7.4 > > > > > -- > Dr. David Alan Gilbert / [email protected] / Manchester, UK
