On Thu, Dec 01, 2022 at 07:13:03AM +0100, Markus Armbruster wrote: > When argument @time isn't 'now' or 'never', we parse it as an integer, > optionally prefixed with '+'. If parsing fails, we silently assume > zero. Report an error and fail instead. > > While there, use qemu_strtou64() instead of strtoull() so > checkpatch.pl won't complain. > > Aside: encoding numbers in strings is bad QMP practice. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > monitor/qmp-cmds.c | 17 +++++++++++++++-- > 1 file changed, 15 insertions(+), 2 deletions(-)
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
