On Wed, Dec 08, 2021 at 06:41:22PM +0000, Dr. David Alan Gilbert wrote:
> * Peter Xu (pet...@redhat.com) wrote:
> > It'll be easier to read the name rather than index of sub-cmd when
> > debugging.
> >
> > Signed-off-by: Peter Xu <pet...@redhat.com>
> > ---
> > migration/savevm.c | 2 +-
> > migration/trace-events | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/migration/savevm.c b/migration/savevm.c
> > index d59e976d50..17b8e25e00 100644
> > --- a/migration/savevm.c
> > +++ b/migration/savevm.c
> > @@ -2271,7 +2271,7 @@ static int loadvm_process_command(QEMUFile *f)
> > return qemu_file_get_error(f);
> > }
> >
> > - trace_loadvm_process_command(cmd, len);
> > + trace_loadvm_process_command(mig_cmd_args[cmd].name, len);
> > if (cmd >= MIG_CMD_MAX || cmd == MIG_CMD_INVALID) {
>
> No! you can't do that name lookup before the limit check.
Heh, right. :)
I guess it shouldn't matter in reality as we don't worry too much on untrusted
or uncompatible src qemu, but it's very reasonable to fix it.
Thanks!
--
Peter Xu
