svn commit: r287796 - stable/9/usr.bin/tftp

Xin LI delphij at FreeBSD.org
Mon Sep 14 18:58:31 UTC 2015


Author: delphij
Date: Mon Sep 14 18:58:30 2015
New Revision: 287796
URL: https://svnweb.freebsd.org/changeset/base/287796

Log:
  MFC r287320:
   - uri is expected to be nul-terminated (strchr used later),
     so use strlcpy instead of strncpy.
   - replace the other two cases of strncpy+\0 with strlcpy.

Modified:
  stable/9/usr.bin/tftp/main.c
Directory Properties:
  stable/9/usr.bin/tftp/   (props changed)

Modified: stable/9/usr.bin/tftp/main.c
==============================================================================
--- stable/9/usr.bin/tftp/main.c	Mon Sep 14 18:57:50 2015	(r287795)
+++ stable/9/usr.bin/tftp/main.c	Mon Sep 14 18:58:30 2015	(r287796)
@@ -223,7 +223,7 @@ urihandling(char *URI)
 	char	line[MAXLINE];
 	int	i;
 
-	strncpy(uri, URI, ARG_MAX);
+	strlcpy(uri, URI, ARG_MAX);
 	host = uri + 7;
 
 	if ((s = strchr(host, '/')) == NULL) {
@@ -320,11 +320,10 @@ setpeer0(char *host, const char *lport)
 		/* res->ai_addr <= sizeof(peeraddr) is guaranteed */
 		memcpy(&peer_sock, res->ai_addr, res->ai_addrlen);
 		if (res->ai_canonname) {
-			(void) strncpy(hostname, res->ai_canonname,
+			(void) strlcpy(hostname, res->ai_canonname,
 				sizeof(hostname));
 		} else
-			(void) strncpy(hostname, host, sizeof(hostname));
-		hostname[sizeof(hostname)-1] = 0;
+			(void) strlcpy(hostname, host, sizeof(hostname));
 		connected = 1;
 	}
 


More information about the svn-src-stable-9 mailing list