Avoid overflowing `fname' if the file name prefix given on the command

line is too long. Obtained from: OpenBSD
2002-05-27 04:59:46 +00:00 · 2002-05-27 04:59:46 +00:00 · a62d4b84fd
commit a62d4b84fd
parent a32a4e42ac
1 changed files with 2 additions and 1 deletions
--- a/usr.bin/split/split.c
+++ b/usr.bin/split/split.c
@ -147,7 +147,8 @@ main(argc, argv)
 			++argv;
 		}
 	if (*argv != NULL)			/* File name prefix. */
-		(void)strcpy(fname, *argv++);
+		if (strlcpy(fname, *argv++, sizeof(fname)) >= sizeof(fname))
+			errx(EX_USAGE, "file name prefix is too long");
 	if (*argv != NULL)
 		usage();