diff options
author | Lucas De Marchi <lucas.demarchi@intel.com> | 2019-06-05 10:44:48 -0700 |
---|---|---|
committer | Lucas De Marchi <lucas.demarchi@intel.com> | 2019-06-06 09:48:47 -0700 |
commit | 94a28f391f77f676ab5d412c17d068f970dd4f78 (patch) | |
tree | e88c93482fdd3fb0d5c70cba8438e057db612767 /lib/igt_core.c | |
parent | 3e500bf81d03c8b955c82c89d82fb20ee2b5d073 (diff) |
lib/igt_core: 0 is a valid val for long options
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@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'lib/igt_core.c')
-rw-r--r-- | lib/igt_core.c | 5 |
1 files 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); } |