commit f46bfdbfc8f95cf65d7818ef68a801e063c40332
author Markus Armbruster <armbru@redhat.com> Tue Feb 21 21:14:07 2017 +0100
committer Markus Armbruster <armbru@redhat.com> Thu Feb 23 20:35:36 2017 +0100
tree 3f4d22f4a161acadbfe44219f6a493145c49502a
parent f17fd4fdf0df3d2f3444399d04c38d22b9a3e1b7

util/cutils: Change qemu_strtosz*() from int64_t to uint64_t

This will permit its use in parse_option_size().

Cc: Dr. David Alan Gilbert <dgilbert@redhat.com>
Cc: Eduardo Habkost <ehabkost@redhat.com> (maintainer:X86)
Cc: Kevin Wolf <kwolf@redhat.com> (supporter:Block layer core)
Cc: Max Reitz <mreitz@redhat.com> (supporter:Block layer core)
Cc: qemu-block@nongnu.org (open list:Block layer core)
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <1487708048-2131-24-git-send-email-armbru@redhat.com>

monitor.c

diff --git a/monitor.c b/monitor.c
index 8baeaf0..4ac2702 100644
--- a/monitor.c
+++ b/monitor.c
@@ -2800,7 +2800,7 @@
         case 'o':
             {
                 int ret;
-                int64_t val;
+                uint64_t val;
                 char *end;
 
                 while (qemu_isspace(*p)) {
@@ -2813,7 +2813,7 @@
                     }
                 }
                 ret = qemu_strtosz_MiB(p, &end, &val);
-                if (ret < 0) {
+                if (ret < 0 || val > INT64_MAX) {
                     monitor_printf(mon, "invalid size\n");
                     goto fail;
                 }