d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

o Check that -w width is not above maximum. Use DWIDTH instead of

Browse Source 
hardcoded constant.

Reported by:	Gruzicki Wlodek on BugTraq
Obtained from:	NetBSD, OpenBSD
MFC after:	1 week
This commit is contained in:
Maxim Konovalov 2006-11-22 21:05:17 +00:00
parent a8ebcb1bc0
commit 0d4d06c9d7
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
usr.bin/banner/banner.c
View File

@ -1045,7 +1045,7 @@ main(int argc, char *argv[])
break;
case 'w':
width = atoi(optarg);
if (width <= 0)
if (width <= 0 || width > DWIDTH)
errx(1, "illegal argument for -w option");
break;
case '?':
@ -1056,7 +1056,7 @@ main(int argc, char *argv[])
argv += optind;
for (i = 0; i < width; i++) {
j = i * 132 / width;
j = i * DWIDTH / width;
print[j] = 1;
}