svn commit: r286270 - stable/9/usr.bin/wall

Pedro F. Giffuni pfg at FreeBSD.org
Tue Aug 4 03:07:07 UTC 2015


Author: pfg
Date: Tue Aug  4 03:07:06 2015
New Revision: 286270
URL: https://svnweb.freebsd.org/changeset/base/286270

Log:
  MFC	r286102:
  Buffer overflow in wall(1).
  
  This affected syslogd, wall and talkd.
  Detected by FORTIFY_SOURCE GSoC (with clang).
  
  Submitted by:		Oliver Pinter
  Differential Revision:	https://reviews.freebsd.org/D3254
  Reviewed by:		delphij, jmg

Modified:
  stable/9/usr.bin/wall/ttymsg.c
Directory Properties:
  stable/9/usr.bin/wall/   (props changed)

Modified: stable/9/usr.bin/wall/ttymsg.c
==============================================================================
--- stable/9/usr.bin/wall/ttymsg.c	Tue Aug  4 03:06:23 2015	(r286269)
+++ stable/9/usr.bin/wall/ttymsg.c	Tue Aug  4 03:07:06 2015	(r286270)
@@ -62,7 +62,7 @@ ttymsg(struct iovec *iov, int iovcnt, co
 	struct iovec localiov[7];
 	ssize_t left, wret;
 	int cnt, fd;
-	static char device[MAXNAMLEN] = _PATH_DEV;
+	char device[MAXNAMLEN] = _PATH_DEV;
 	static char errbuf[1024];
 	char *p;
 	int forked;
@@ -71,8 +71,8 @@ ttymsg(struct iovec *iov, int iovcnt, co
 	if (iovcnt > (int)(sizeof(localiov) / sizeof(localiov[0])))
 		return ("too many iov's (change code in wall/ttymsg.c)");
 
+	strlcat(device, line, sizeof(device));
 	p = device + sizeof(_PATH_DEV) - 1;
-	strlcpy(p, line, sizeof(device));
 	if (strncmp(p, "pts/", 4) == 0)
 		p += 4;
 	if (strchr(p, '/') != NULL) {


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