[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 for-5.1 4/9] qemu-option: Fix has_help_option()'s sloppy parsi
From: |
Markus Armbruster |
Subject: |
[PATCH v2 for-5.1 4/9] qemu-option: Fix has_help_option()'s sloppy parsing |
Date: |
Wed, 15 Apr 2020 09:49:22 +0200 |
has_help_option() uses its own parser. It's inconsistent with
qemu_opts_parse(), as demonstrated by test-qemu-opts case
/qemu-opts/has_help_option. Fix by reusing the common parser.
Signed-off-by: Markus Armbruster <address@hidden>
---
tests/test-qemu-opts.c | 4 ++--
util/qemu-option.c | 39 +++++++++++++++++++--------------------
2 files changed, 21 insertions(+), 22 deletions(-)
diff --git a/tests/test-qemu-opts.c b/tests/test-qemu-opts.c
index 8ff97268d8..77c944c4aa 100644
--- a/tests/test-qemu-opts.c
+++ b/tests/test-qemu-opts.c
@@ -749,8 +749,8 @@ static void test_has_help_option(void)
{ "a=0,?,b", true, true, true },
{ "help,b=1", true, true, false },
{ "?,b=1", true, true, false },
- { "a,b,,help", false /* BUG */, true, true },
- { "a,b,,?", false /* BUG */, true, true },
+ { "a,b,,help", true, true, true },
+ { "a,b,,?", true, true, true },
};
int i;
QemuOpts *opts;
diff --git a/util/qemu-option.c b/util/qemu-option.c
index d2956082bd..0abf26b61f 100644
--- a/util/qemu-option.c
+++ b/util/qemu-option.c
@@ -165,26 +165,6 @@ void parse_option_size(const char *name, const char *value,
*ret = size;
}
-bool has_help_option(const char *param)
-{
- const char *p = param;
- bool result = false;
-
- while (*p && !result) {
- char *value;
-
- p = get_opt_value(p, &value);
- if (*p) {
- p++;
- }
-
- result = is_help_option(value);
- g_free(value);
- }
-
- return result;
-}
-
bool is_valid_option_list(const char *p)
{
char *value = NULL;
@@ -890,6 +870,25 @@ static char *opts_parse_id(const char *params)
return NULL;
}
+bool has_help_option(const char *params)
+{
+ const char *p;
+ char *name, *value;
+ bool ret;
+
+ for (p = params; *p;) {
+ p = get_opt_name_value(p, NULL, &name, &value);
+ ret = is_help_option(name);
+ g_free(name);
+ g_free(value);
+ if (ret) {
+ return true;
+ }
+ }
+
+ return false;
+}
+
/**
* Store options parsed from @params into @opts.
* If @firstname is non-null, the first key=value in @params may omit
--
2.21.1
- [PATCH v2 for-5.1 0/9] qemu-option: Fix corner cases and clean up, Markus Armbruster, 2020/04/15
- [PATCH v2 for-5.1 3/9] qemu-option: Fix sloppy recognition of "id=..." after ", , ", Markus Armbruster, 2020/04/15
- [PATCH v2 for-5.1 4/9] qemu-option: Fix has_help_option()'s sloppy parsing,
Markus Armbruster <=
- [PATCH v2 for-5.1 6/9] qemu-option: Avoid has_help_option() in qemu_opts_parse_noisily(), Markus Armbruster, 2020/04/15
- [PATCH v2 for-5.1 5/9] test-qemu-opts: Simplify test_has_help_option() after bug fix, Markus Armbruster, 2020/04/15
- [PATCH v2 for-5.1 1/9] tests-qemu-opts: Cover has_help_option(), qemu_opt_has_help_opt(), Markus Armbruster, 2020/04/15
- [PATCH v2 for-5.1 7/9] qemu-img: Factor out accumulate_options() helper, Markus Armbruster, 2020/04/15
- [PATCH v2 for-5.1 2/9] qemu-options: Factor out get_opt_name_value() helper, Markus Armbruster, 2020/04/15
- [PATCH v2 for-5.1 9/9] qemu-img: Reject broken -o "", Markus Armbruster, 2020/04/15
- [PATCH v2 for-5.1 8/9] qemu-img: Move is_valid_option_list() to qemu-img.c and rewrite, Markus Armbruster, 2020/04/15