Loading...
stdlib/FreeBSD/realpath.c Libc-1725.40.4 Libc-825.26
--- Libc/Libc-1725.40.4/stdlib/FreeBSD/realpath.c
+++ Libc/Libc-825.26/stdlib/FreeBSD/realpath.c
@@ -45,7 +45,6 @@
 #include <sys/vnode.h>
 #include "un-namespace.h"
 
-
 struct attrs {
 	u_int32_t len;
 	attrreference_t name;
@@ -56,7 +55,7 @@
 };
 
 #ifndef BUILDING_VARIANT
-__private_extern__ const struct attrlist _rp_alist = {
+__private_extern__ struct attrlist _rp_alist = {
 	ATTR_BIT_MAP_COUNT,
 	0,
 	ATTR_CMN_NAME | ATTR_CMN_DEVID | ATTR_CMN_OBJTYPE | ATTR_CMN_OBJID,
@@ -66,7 +65,7 @@
 	0,
 };
 #else /* BUILDING_VARIANT */
-extern const struct attrlist _rp_alist;
+__private_extern__ struct attrlist _rp_alist;
 #endif /* BUILDING_VARIANT */
 
 extern char * __private_getcwd(char *, size_t, int);
@@ -83,11 +82,10 @@
 {
 	struct attrs attrs;
 	struct stat sb;
-	char *p, *q;
-	size_t left_len, resolved_len, save_resolved_len, next_token_len;
+	char *p, *q, *s;
+	size_t left_len, resolved_len, save_resolved_len;
 	unsigned symlinks;
-	int serrno, useattrs, islink;
-	ssize_t slen;
+	int serrno, slen, useattrs, islink;
 	char left[PATH_MAX], next_token[PATH_MAX], symlink[PATH_MAX];
 	dev_t dev, lastdev;
 	struct statfs sfs;
@@ -106,7 +104,6 @@
 		return (NULL);
 	}
 #endif /* __DARWIN_UNIX03 */
-
 	/*
 	 * Extension to the standard; if inresolved == NULL, allocate memory
 	 */
@@ -174,18 +171,16 @@
 		 * and its length.
 		 */
 		p = strchr(left, '/');
-		next_token_len = p ? p - left : left_len;
-		memcpy(next_token, left, next_token_len);
-		next_token[next_token_len] = '\0';
-
-		if (p != NULL) {
-			left_len -= next_token_len + 1;
-			memmove(left, p + 1, left_len + 1);
-		} else {
-			left[0] = '\0';
-			left_len = 0;
-		}
-
+		s = p ? p : left + left_len;
+		if (s - left >= sizeof(next_token)) {
+			errno = ENAMETOOLONG;
+			goto error_return;
+		}
+		memcpy(next_token, left, s - left);
+		next_token[s - left] = '\0';
+		left_len -= s - left;
+		if (p != NULL)
+			memmove(left, s + 1, left_len + 1);
 		if (resolved[resolved_len - 1] != '/') {
 			if (resolved_len + 1 >= PATH_MAX) {
 				errno = ENAMETOOLONG;
@@ -194,12 +189,11 @@
 			resolved[resolved_len++] = '/';
 			resolved[resolved_len] = '\0';
 		}
-		if (next_token[0] == '\0') {
-			/* Handle consequential slashes. */
+		if (next_token[0] == '\0')
 			continue;
-		} else if (strcmp(next_token, ".") == 0) {
+		else if (strcmp(next_token, ".") == 0)
 			continue;
-		} else if (strcmp(next_token, "..") == 0) {
+		else if (strcmp(next_token, "..") == 0) {
 			/*
 			 * Strip the last path component except when we have
 			 * single "/"
@@ -230,7 +224,7 @@
 			errno = ENAMETOOLONG;
 			goto error_return;
 		}
-		if (getattrlist(resolved, (void *)&_rp_alist, &attrs, sizeof(attrs), FSOPT_NOFOLLOW) == 0) {
+		if (getattrlist(resolved, &_rp_alist, &attrs, sizeof(attrs), FSOPT_NOFOLLOW) == 0) {
 			useattrs = 1;
 			islink = (attrs.type == VLNK);
 			dev = attrs.dev;
@@ -302,14 +296,8 @@
 				errno = ELOOP;
 				goto error_return;
 			}
-			slen = readlink(resolved, symlink, sizeof(symlink));
-			if (slen <= 0 || slen >= sizeof(symlink)) {
-				if (slen < 0)
-					; /* keep errno from readlink(2) call */
-				else if (slen == 0)
-					errno = ENOENT;
-				else
-					errno = ENAMETOOLONG;
+			slen = readlink(resolved, symlink, sizeof(symlink) - 1);
+			if (slen < 0) {
 				goto error_return;
 			}
 			symlink[slen] = '\0';
@@ -317,8 +305,9 @@
 				resolved[1] = 0;
 				resolved_len = 1;
 				lastdev = rootdev;
-			} else {
+			} else if (resolved_len > 1) {
 				/* Strip the last path component. */
+				resolved[resolved_len - 1] = '\0';
 				q = strrchr(resolved, '/') + 1;
 				*q = '\0';
 				resolved_len = q - resolved;
@@ -338,8 +327,8 @@
 					symlink[slen] = '/';
 					symlink[slen + 1] = 0;
 				}
-				left_len = strlcat(symlink, left, sizeof(symlink));
-				if (left_len >= sizeof(symlink)) {
+				left_len = strlcat(symlink, left, sizeof(left));
+				if (left_len >= sizeof(left)) {
 					errno = ENAMETOOLONG;
 					goto error_return;
 				}