Author: imp
Date: Sat Jan  6 12:46:04 2018
New Revision: 327619
URL: https://svnweb.freebsd.org/changeset/base/327619

Log:
  The source strings are from the password database which guarantees
  that the data going into it is sane.  Out of an abundance of caution,
  limit the string copies to prevent an overflow.
  
  CID: 1019035

Modified:
  head/lib/libc/net/rcmd.c

Modified: head/lib/libc/net/rcmd.c
==============================================================================
--- head/lib/libc/net/rcmd.c    Sat Jan  6 12:45:59 2018        (r327618)
+++ head/lib/libc/net/rcmd.c    Sat Jan  6 12:46:04 2018        (r327619)
@@ -457,8 +457,8 @@ again:
                first = 0;
                if ((pwd = getpwnam(luser)) == NULL)
                        return (-1);
-               (void)strcpy(pbuf, pwd->pw_dir);
-               (void)strcat(pbuf, "/.rhosts");
+               (void)strlcpy(pbuf, pwd->pw_dir, sizeof(pbuf));
+               (void)strlcat(pbuf, "/.rhosts", sizeof(pbuf));
 
                /*
                 * Change effective uid while opening .rhosts.  If root and
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to