yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone
9ae3a8
From 00e5b1ffe79264929ea0dab871e5e797ab85497f Mon Sep 17 00:00:00 2001
9ae3a8
From: Max Reitz <mreitz@redhat.com>
9ae3a8
Date: Mon, 19 Aug 2013 11:09:45 +0200
9ae3a8
Subject: qemu-img: Error out for excess arguments
9ae3a8
9ae3a8
RH-Author: Max Reitz <mreitz@redhat.com>
9ae3a8
Message-id: <1376910585-8875-1-git-send-email-mreitz@redhat.com>
9ae3a8
Patchwork-id: 53597
9ae3a8
O-Subject: [RHEL-7 qemu-kvm PATCH] qemu-img: Error out for excess arguments
9ae3a8
Bugzilla: 992935
9ae3a8
RH-Acked-by: Markus Armbruster <armbru@redhat.com>
9ae3a8
RH-Acked-by: Kevin Wolf <kwolf@redhat.com>
9ae3a8
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
9ae3a8
9ae3a8
From: Kevin Wolf <kwolf@redhat.com>
9ae3a8
9ae3a8
Don't silently ignore excess arguments at the end of the command line,
9ae3a8
but error out instead. This can catch typos like 'resize test.img + 1G',
9ae3a8
which doesn't increase the image size by 1G as intended, but truncates
9ae3a8
the image to 1G. Even for less dangerous commands, the old behaviour is
9ae3a8
confusing.
9ae3a8
9ae3a8
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
9ae3a8
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
9ae3a8
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
9ae3a8
(cherry picked from commit fc11eb26cee7e3621645dd40cd9de944201f590b)
9ae3a8
Signed-off-by: Max Reitz <mreitz@redhat.com>
9ae3a8
9ae3a8
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=992935
9ae3a8
Brew: https://brewweb.devel.redhat.com/taskinfo?taskID=6181030
9ae3a8
9ae3a8
diff --git a/qemu-img.c b/qemu-img.c
9ae3a8
index cd096a1..c20ff66 100644
9ae3a8
--- a/qemu-img.c
9ae3a8
+++ b/qemu-img.c
9ae3a8
@@ -393,6 +393,9 @@ static int img_create(int argc, char **argv)
9ae3a8
         }
9ae3a8
         img_size = (uint64_t)sval;
9ae3a8
     }
9ae3a8
+    if (optind != argc) {
9ae3a8
+        help();
9ae3a8
+    }
9ae3a8
 
9ae3a8
     if (options && is_help_option(options)) {
9ae3a8
         return print_block_option_help(filename, fmt);
9ae3a8
@@ -570,7 +573,7 @@ static int img_check(int argc, char **argv)
9ae3a8
             break;
9ae3a8
         }
9ae3a8
     }
9ae3a8
-    if (optind >= argc) {
9ae3a8
+    if (optind != argc - 1) {
9ae3a8
         help();
9ae3a8
     }
9ae3a8
     filename = argv[optind++];
9ae3a8
@@ -681,7 +684,7 @@ static int img_commit(int argc, char **argv)
9ae3a8
             break;
9ae3a8
         }
9ae3a8
     }
9ae3a8
-    if (optind >= argc) {
9ae3a8
+    if (optind != argc - 1) {
9ae3a8
         help();
9ae3a8
     }
9ae3a8
     filename = argv[optind++];
9ae3a8
@@ -927,7 +930,7 @@ static int img_compare(int argc, char **argv)
9ae3a8
     }
9ae3a8
 
9ae3a8
 
9ae3a8
-    if (optind > argc - 2) {
9ae3a8
+    if (optind != argc - 2) {
9ae3a8
         help();
9ae3a8
     }
9ae3a8
     filename1 = argv[optind++];
9ae3a8
@@ -1882,7 +1885,7 @@ static int img_info(int argc, char **argv)
9ae3a8
             break;
9ae3a8
         }
9ae3a8
     }
9ae3a8
-    if (optind >= argc) {
9ae3a8
+    if (optind != argc - 1) {
9ae3a8
         help();
9ae3a8
     }
9ae3a8
     filename = argv[optind++];
9ae3a8
@@ -1983,7 +1986,7 @@ static int img_snapshot(int argc, char **argv)
9ae3a8
         }
9ae3a8
     }
9ae3a8
 
9ae3a8
-    if (optind >= argc) {
9ae3a8
+    if (optind != argc - 1) {
9ae3a8
         help();
9ae3a8
     }
9ae3a8
     filename = argv[optind++];
9ae3a8
@@ -2094,7 +2097,7 @@ static int img_rebase(int argc, char **argv)
9ae3a8
         progress = 0;
9ae3a8
     }
9ae3a8
 
9ae3a8
-    if ((optind >= argc) || (!unsafe && !out_baseimg)) {
9ae3a8
+    if ((optind != argc - 1) || (!unsafe && !out_baseimg)) {
9ae3a8
         help();
9ae3a8
     }
9ae3a8
     filename = argv[optind++];
9ae3a8
@@ -2373,7 +2376,7 @@ static int img_resize(int argc, char **argv)
9ae3a8
             break;
9ae3a8
         }
9ae3a8
     }
9ae3a8
-    if (optind >= argc) {
9ae3a8
+    if (optind != argc - 1) {
9ae3a8
         help();
9ae3a8
     }
9ae3a8
     filename = argv[optind++];