Check the whole value

zdohnal · zdohnal · commit a0c075b21a67 · 2026-03-16T16:23:32.000+01:00
diff --git a/scheduler/conf.c b/scheduler/conf.c
@@ -3026,12 +3026,15 @@ parse_variable(
 	  return (0);
 	}
 
-	if (!isdigit(*value & 255))
+	while (*value)
 	{
-	  cupsdLogMessage(CUPSD_LOG_ERROR,
-			  "Bad integer value for %s on line %d of %s.",
-			  line, linenum, filename);
-	  return (0);
+	  if (!isdigit(*value++))
+	  {
+	    cupsdLogMessage(CUPSD_LOG_ERROR,
+			    "Bad integer value for %s on line %d of %s.",
+			    line, linenum, filename);
+	    return (0);
+	  }
 	}
 
 	int n = (int)strtol(value, NULL, 10);

Original file line number	Diff line number	Diff line change
`@@ -3026,12 +3026,15 @@ parse_variable(`
`3026`	`3026`	`return (0);`
`3027`	`3027`	`}`
`3028`	`3028`
`3029`		`- if (!isdigit(*value & 255))`
	`3029`	`+ while (*value)`
`3030`	`3030`	`{`
`3031`		`- cupsdLogMessage(CUPSD_LOG_ERROR,`
`3032`		`- "Bad integer value for %s on line %d of %s.",`
`3033`		`- line, linenum, filename);`
`3034`		`- return (0);`
	`3031`	`+ if (!isdigit(*value++))`
	`3032`	`+ {`
	`3033`	`+ cupsdLogMessage(CUPSD_LOG_ERROR,`
	`3034`	`+ "Bad integer value for %s on line %d of %s.",`
	`3035`	`+ line, linenum, filename);`
	`3036`	`+ return (0);`
	`3037`	`+ }`
`3035`	`3038`	`}`
`3036`	`3039`
`3037`	`3040`	`int n = (int)strtol(value, NULL, 10);`