The -s option was broken (missing braces around an else clause).

Noticed by: Thomas Stromberg <tstromberg@rtci.com> Obtained from: OpenBSD
1999-12-04 03:19:15 +00:00 · 1999-12-04 03:19:15 +00:00 · d68e2acd95
commit d68e2acd95
parent b26a5ea6fe
1 changed files with 7 additions and 5 deletions
--- a/usr.bin/pr/pr.c
+++ b/usr.bin/pr/pr.c
@ -1709,12 +1709,14 @@ setup(argc, argv)
 			++sflag;
 			if (eoptarg == NULL)
 				schar = SCHAR;
-			else
+			else {
 				schar = *eoptarg++;
-			if (*eoptarg != '\0') {
-				(void)fprintf(err,
-				      "pr: invalid value for -s %s\n", eoptarg);
-				return(1);
+				if (*eoptarg != '\0') {
+					(void)fprintf(err,
+					    "pr: invalid value for -s %s\n",
+					    eoptarg);
+					return(1);
+				}
 			}
 			break;
 		case 't':