svn commit: r355260 - head/libexec/getty

[Xin LI]

Author: delphij
Date: Sun Dec  1 08:04:22 2019
New Revision: 355260
URL: https://svnweb.freebsd.org/changeset/base/355260

Log:
  Simplify code with strlcpy/strlcat.
  
  MFC after:	2 weeks

Modified:
  head/libexec/getty/main.c
  head/libexec/getty/subr.c

Modified: head/libexec/getty/main.c
==============================================================================
--- head/libexec/getty/main.c	Sun Dec  1 07:44:21 2019	(r355259)
+++ head/libexec/getty/main.c	Sun Dec  1 08:04:22 2019	(r355260)
@@ -97,7 +97,6 @@ static int crmod, digit, lower, upper;
 
 char	hostname[MAXHOSTNAMELEN];
 static char	name[MAXLOGNAME*3];
-static char	dev[] = _PATH_DEV;
 static char	ttyn[32];
 
 #define	OBUFSIZ		128
@@ -218,8 +217,8 @@ main(int argc, char *argv[])
 	if (argc <= 2 || strcmp(argv[2], "-") == 0)
 	    strcpy(ttyn, ttyname(STDIN_FILENO));
 	else {
-	    strcpy(ttyn, dev);
-	    strncat(ttyn, argv[2], sizeof(ttyn)-sizeof(dev));
+	    strcpy(ttyn, _PATH_DEV);
+	    strlcat(ttyn, argv[2], sizeof(ttyn));
 	    if (strcmp(argv[0], "+") != 0) {
 		chown(ttyn, 0, 0);
 		chmod(ttyn, 0600);

Modified: head/libexec/getty/subr.c
==============================================================================
--- head/libexec/getty/subr.c	Sun Dec  1 07:44:21 2019	(r355259)
+++ head/libexec/getty/subr.c	Sun Dec  1 08:04:22 2019	(r355260)
@@ -89,10 +89,8 @@ gettable(const char *name, char *buf)
 					l = 2;
 				else
 					l = strlen(sp->value) + 1;
-				if ((p = malloc(l)) != NULL) {
-					strncpy(p, sp->value, l);
-					p[l-1] = '\0';
-				}
+				if ((p = malloc(l)) != NULL)
+					strlcpy(p, sp->value, l);
 				/*
 				 * replace, even if NULL, else we'll
 				 * have problems with free()ing static mem