plugins/hotblocks: Added correct boolean argument parsing
authorMahmoud Mandour <ma.mandourr@gmail.com>
Fri, 30 Jul 2021 13:58:08 +0000 (15:58 +0200)
committerAlex Bennée <alex.bennee@linaro.org>
Thu, 2 Sep 2021 10:29:34 +0000 (11:29 +0100)
Signed-off-by: Mahmoud Mandour <ma.mandourr@gmail.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20210730135817.17816-5-ma.mandourr@gmail.com>

contrib/plugins/hotblocks.c

index 4b08340143242a565b9aadd5d36af99aaea05ea7..062200a7a42351283dbc3e499a07b1402d48ca8b 100644 (file)
@@ -133,8 +133,18 @@ QEMU_PLUGIN_EXPORT
 int qemu_plugin_install(qemu_plugin_id_t id, const qemu_info_t *info,
                         int argc, char **argv)
 {
-    if (argc && strcmp(argv[0], "inline") == 0) {
-        do_inline = true;
+    for (int i = 0; i < argc; i++) {
+        char *opt = argv[i];
+        g_autofree char **tokens = g_strsplit(opt, "=", 2);
+        if (g_strcmp0(tokens[0], "inline") == 0) {
+            if (!qemu_plugin_bool_parse(tokens[0], tokens[1], &do_inline)) {
+                fprintf(stderr, "boolean argument parsing failed: %s\n", opt);
+                return -1;
+            }
+        } else {
+            fprintf(stderr, "option parsing failed: %s\n", opt);
+            return -1;
+        }
     }
 
     plugin_init();