cryptospore / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone
9ae3a8
From c73b87743086200c3f9ad485dcd4e71bdee27881 Mon Sep 17 00:00:00 2001
9ae3a8
From: John Snow <jsnow@redhat.com>
9ae3a8
Date: Mon, 23 Nov 2015 17:38:26 +0100
9ae3a8
Subject: [PATCH 07/27] qemu-io: Factor out qemuio_command
9ae3a8
9ae3a8
RH-Author: John Snow <jsnow@redhat.com>
9ae3a8
Message-id: <1448300320-7772-8-git-send-email-jsnow@redhat.com>
9ae3a8
Patchwork-id: 68438
9ae3a8
O-Subject: [RHEL-7.3 qemu-kvm PATCH v2 07/21] qemu-io: Factor out qemuio_command
9ae3a8
Bugzilla: 1272523
9ae3a8
RH-Acked-by: Thomas Huth <thuth@redhat.com>
9ae3a8
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
9ae3a8
RH-Acked-by: Max Reitz <mreitz@redhat.com>
9ae3a8
9ae3a8
From: Kevin Wolf <kwolf@redhat.com>
9ae3a8
9ae3a8
It's duplicated code. Move it to qemu-io-cmds.c because it's not
9ae3a8
dependent on any static data of the qemu-io tool.
9ae3a8
9ae3a8
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
9ae3a8
Reviewed-by: Eric Blake <eblake@redhat.com>
9ae3a8
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
9ae3a8
(cherry picked from commit dd5832967ac3fe96bd5bf9f199639176998ead69)
9ae3a8
Signed-off-by: John Snow <jsnow@redhat.com>
9ae3a8
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
9ae3a8
---
9ae3a8
 cmd.c          | 43 +++++--------------------------------------
9ae3a8
 cmd.h          |  3 ++-
9ae3a8
 qemu-io-cmds.c | 24 ++++++++++++++++++++++++
9ae3a8
 3 files changed, 31 insertions(+), 39 deletions(-)
9ae3a8
9ae3a8
diff --git a/cmd.c b/cmd.c
9ae3a8
index d501aab..7ae978f 100644
9ae3a8
--- a/cmd.c
9ae3a8
+++ b/cmd.c
9ae3a8
@@ -138,28 +138,11 @@ static char *get_prompt(void);
9ae3a8
 
9ae3a8
 void command_loop(void)
9ae3a8
 {
9ae3a8
-    int c, i, done = 0, fetchable = 0, prompted = 0;
9ae3a8
+    int i, done = 0, fetchable = 0, prompted = 0;
9ae3a8
     char *input;
9ae3a8
-    char **v;
9ae3a8
-    const cmdinfo_t *ct;
9ae3a8
 
9ae3a8
     for (i = 0; !done && i < ncmdline; i++) {
9ae3a8
-        input = strdup(cmdline[i]);
9ae3a8
-        if (!input) {
9ae3a8
-            fprintf(stderr, _("cannot strdup command '%s': %s\n"),
9ae3a8
-                    cmdline[i], strerror(errno));
9ae3a8
-            exit(1);
9ae3a8
-        }
9ae3a8
-        v = breakline(input, &c);
9ae3a8
-        if (c) {
9ae3a8
-            ct = find_command(v[0]);
9ae3a8
-            if (ct) {
9ae3a8
-                done = command(ct, c, v);
9ae3a8
-            } else {
9ae3a8
-                fprintf(stderr, _("command \"%s\" not found\n"), v[0]);
9ae3a8
-            }
9ae3a8
-	}
9ae3a8
-        doneline(input, v);
9ae3a8
+        done = qemuio_command(cmdline[i]);
9ae3a8
     }
9ae3a8
     if (cmdline) {
9ae3a8
         g_free(cmdline);
9ae3a8
@@ -179,20 +162,13 @@ void command_loop(void)
9ae3a8
         if (!fetchable) {
9ae3a8
             continue;
9ae3a8
         }
9ae3a8
+
9ae3a8
         input = fetchline();
9ae3a8
         if (input == NULL) {
9ae3a8
             break;
9ae3a8
         }
9ae3a8
-        v = breakline(input, &c);
9ae3a8
-        if (c) {
9ae3a8
-            ct = find_command(v[0]);
9ae3a8
-            if (ct) {
9ae3a8
-                done = command(ct, c, v);
9ae3a8
-            } else {
9ae3a8
-                fprintf(stderr, _("command \"%s\" not found\n"), v[0]);
9ae3a8
-            }
9ae3a8
-        }
9ae3a8
-        doneline(input, v);
9ae3a8
+        done = qemuio_command(input);
9ae3a8
+        free(input);
9ae3a8
 
9ae3a8
         prompted = 0;
9ae3a8
         fetchable = 0;
9ae3a8
@@ -328,15 +304,6 @@ char **breakline(char *input, int *count)
9ae3a8
     return rval;
9ae3a8
 }
9ae3a8
 
9ae3a8
-void
9ae3a8
-doneline(
9ae3a8
-	char	*input,
9ae3a8
-	char	**vec)
9ae3a8
-{
9ae3a8
-	free(input);
9ae3a8
-	free(vec);
9ae3a8
-}
9ae3a8
-
9ae3a8
 #define EXABYTES(x)	((long long)(x) << 60)
9ae3a8
 #define PETABYTES(x)	((long long)(x) << 50)
9ae3a8
 #define TERABYTES(x)	((long long)(x) << 40)
9ae3a8
diff --git a/cmd.h b/cmd.h
9ae3a8
index ccf6336..d676408 100644
9ae3a8
--- a/cmd.h
9ae3a8
+++ b/cmd.h
9ae3a8
@@ -59,7 +59,6 @@ int command(const cmdinfo_t *ci, int argc, char **argv);
9ae3a8
 
9ae3a8
 /* from input.h */
9ae3a8
 char **breakline(char *input, int *count);
9ae3a8
-void doneline(char *input, char **vec);
9ae3a8
 char *fetchline(void);
9ae3a8
 
9ae3a8
 void cvtstr(double value, char *str, size_t sz);
9ae3a8
@@ -77,4 +76,6 @@ void timestr(struct timeval *tv, char *str, size_t sz, int flags);
9ae3a8
 
9ae3a8
 extern char *progname;
9ae3a8
 
9ae3a8
+bool qemuio_command(const char *cmd);
9ae3a8
+
9ae3a8
 #endif	/* __COMMAND_H__ */
9ae3a8
diff --git a/qemu-io-cmds.c b/qemu-io-cmds.c
9ae3a8
index efa6070..09e4099 100644
9ae3a8
--- a/qemu-io-cmds.c
9ae3a8
+++ b/qemu-io-cmds.c
9ae3a8
@@ -1824,6 +1824,30 @@ static int init_check_command(BlockDriverState *bs, const cmdinfo_t *ct)
9ae3a8
     return 1;
9ae3a8
 }
9ae3a8
 
9ae3a8
+bool qemuio_command(const char *cmd)
9ae3a8
+{
9ae3a8
+    char *input;
9ae3a8
+    const cmdinfo_t *ct;
9ae3a8
+    char **v;
9ae3a8
+    int c;
9ae3a8
+    bool done = false;
9ae3a8
+
9ae3a8
+    input = g_strdup(cmd);
9ae3a8
+    v = breakline(input, &c);
9ae3a8
+    if (c) {
9ae3a8
+        ct = find_command(v[0]);
9ae3a8
+        if (ct) {
9ae3a8
+            done = command(ct, c, v);
9ae3a8
+        } else {
9ae3a8
+            fprintf(stderr, "command \"%s\" not found\n", v[0]);
9ae3a8
+        }
9ae3a8
+    }
9ae3a8
+    g_free(input);
9ae3a8
+    g_free(v);
9ae3a8
+
9ae3a8
+    return done;
9ae3a8
+}
9ae3a8
+
9ae3a8
 static void __attribute((constructor)) init_qemuio_commands(void)
9ae3a8
 {
9ae3a8
     /* initialize commands */
9ae3a8
-- 
9ae3a8
1.8.3.1
9ae3a8