[Intel-gfx] [PATCH i-g-t 3/5] lib/igt_core: 0 is a valid val for long options
Lucas De Marchi
lucas.demarchi at intel.com
Wed May 29 23:27:35 UTC 2019
This is usually used by long options when working with enum to set long
option values. So replace the strchr() with a memchr() to take that into
account.
Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
---
lib/igt_core.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/lib/igt_core.c b/lib/igt_core.c
index 814f5c72..a0b7e581 100644
--- a/lib/igt_core.c
+++ b/lib/igt_core.c
@@ -680,6 +680,7 @@ static int common_init(int *argc, char **argv,
};
char *short_opts;
const char *std_short_opts = "h";
+ size_t std_short_opts_len = strlen(std_short_opts);
struct option *combined_opts;
int extra_opt_count;
int all_opt_count;
@@ -713,7 +714,7 @@ static int common_init(int *argc, char **argv,
/* check for conflicts with standard short options */
if (extra_long_opts[extra_opt_count].val != ':'
- && (conflicting_char = strchr(std_short_opts, extra_long_opts[extra_opt_count].val))) {
+ && (conflicting_char = memchr(std_short_opts, extra_long_opts[extra_opt_count].val, std_short_opts_len))) {
igt_critical("Conflicting long and short option 'val' representation between --%s and -%c\n",
extra_long_opts[extra_opt_count].name,
*conflicting_char);
@@ -727,7 +728,7 @@ static int common_init(int *argc, char **argv,
continue;
/* check for conflicts with standard short options */
- if (strchr(std_short_opts, extra_short_opts[i])) {
+ if (memchr(std_short_opts, extra_short_opts[i], std_short_opts_len)) {
igt_critical("Conflicting short option: -%c\n", std_short_opts[i]);
assert(0);
}
--
2.21.0
More information about the Intel-gfx
mailing list