[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