svn commit: r318097 - stable/11/usr.bin/xinstall

[Bryan Drewery]

Author: bdrewery
Date: Tue May  9 19:15:11 2017
New Revision: 318097
URL: https://svnweb.freebsd.org/changeset/base/318097

Log:
  MFC r303450:
  
    Pull a copy of the input string before calling basename() and dirname().

Modified:
  stable/11/usr.bin/xinstall/xinstall.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/usr.bin/xinstall/xinstall.c
==============================================================================
--- stable/11/usr.bin/xinstall/xinstall.c	Tue May  9 19:14:26 2017	(r318096)
+++ stable/11/usr.bin/xinstall/xinstall.c	Tue May  9 19:15:11 2017	(r318097)
@@ -667,7 +667,7 @@ makelink(const char *from_name, const ch
 	}
 
 	if (dolink & LN_RELATIVE) {
-		char *cp, *d, *s;
+		char *to_name_copy, *cp, *d, *s;
 
 		if (*from_name != '/') {
 			/* this is already a relative link */
@@ -685,7 +685,10 @@ makelink(const char *from_name, const ch
 		 * The last component of to_name may be a symlink,
 		 * so use realpath to resolve only the directory.
 		 */
-		cp = dirname(to_name);
+		to_name_copy = strdup(to_name);
+		if (to_name_copy == NULL)
+			err(EX_OSERR, "%s: strdup", to_name);
+		cp = dirname(to_name_copy);
 		if (realpath(cp, dst) == NULL)
 			err(EX_OSERR, "%s: realpath", cp);
 		/* .. and add the last component. */
@@ -693,9 +696,11 @@ makelink(const char *from_name, const ch
 			if (strlcat(dst, "/", sizeof(dst)) > sizeof(dst))
 				errx(1, "resolved pathname too long");
 		}
-		cp = basename(to_name);
+		strcpy(to_name_copy, to_name);
+		cp = basename(to_name_copy);
 		if (strlcat(dst, cp, sizeof(dst)) > sizeof(dst))
 			errx(1, "resolved pathname too long");
+		free(to_name_copy);
 
 		/* Trim common path components. */
 		for (s = src, d = dst; *s == *d; s++, d++)