src/jloptions.c: fix buggy use of strchr (#34432)

StefanKarpinski · KristofferC · commit d1bd7fa15c7b · 2020-04-11T19:32:10.000+02:00
diff --git a/src/jloptions.c b/src/jloptions.c
@@ -283,7 +283,7 @@ JL_DLLEXPORT void jl_parse_opts(int *argcp, char ***argvp)
                             c = o->val;
                             goto restart_switch;
                         }
-                        else if (strchr(shortopts, o->val)) {
+                        else if (o->val <= 0xff && strchr(shortopts, o->val)) {
                             jl_errorf("option `-%c/--%s` is missing an argument", o->val, o->name);
                         }
                         else {

Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,7 @@ JL_DLLEXPORT void jl_parse_opts(int argcp, char **argvp)`
`283`	`283`	`c = o->val;`
`284`	`284`	`goto restart_switch;`
`285`	`285`	`}`
`286`		`- else if (strchr(shortopts, o->val)) {`
	`286`	`+ else if (o->val <= 0xff && strchr(shortopts, o->val)) {`
`287`	`287`	jl_errorf("option `-%c/--%s` is missing an argument", o->val, o->name);
`288`	`288`	`}`
`289`	`289`	`else {`