Message ID | 1321951566-11667-3-git-send-email-armbru@redhat.com |
---|---|
State | New |
Headers | show |
diff --git a/vl.c b/vl.c index f5afed4..b9146cf 100644 --- a/vl.c +++ b/vl.c @@ -953,8 +953,8 @@ static void numa_add(const char *optarg) node_mem[nodenr] = 0; } else { int64_t sval; - sval = strtosz(option, NULL); - if (sval < 0) { + sval = strtosz(option, &endptr); + if (sval < 0 || *endptr) { fprintf(stderr, "qemu: invalid numa mem size: %s\n", optarg); exit(1); }
strtosz_suffix() fails unless the size is followed by 0, whitespace or ','. Useless here, because we need to fail for any junk following the size, even if it starts with whitespace or ','. Check manually. Things like -smp 4 -numa "node,mem=1024,cpus=0-1" -numa "node,mem=1024 cpus=2-3" are now caught. Before, the second -numa's argument was silently interpreted as just "node,mem=1024". Signed-off-by: Markus Armbruster <armbru@redhat.com> --- vl.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)