Blame SOURCES/sysfsutils-2.1.0-get_link.patch

6aa142
diff -upr sysfsutils-2.1.0-old/lib/sysfs_utils.c sysfsutils-2.1.0/lib/sysfs_utils.c
6aa142
--- sysfsutils-2.1.0-old/lib/sysfs_utils.c	2006-08-07 07:08:01.000000000 +0200
6aa142
+++ sysfsutils-2.1.0/lib/sysfs_utils.c	2008-05-13 07:42:50.000000000 +0200
6aa142
@@ -117,84 +117,104 @@ int sysfs_get_link(const char *path, cha
6aa142
 {
6aa142
 	char devdir[SYSFS_PATH_MAX];
6aa142
 	char linkpath[SYSFS_PATH_MAX];
6aa142
-	char temp_path[SYSFS_PATH_MAX];
6aa142
-	char *d = NULL, *s = NULL;
6aa142
-	int slashes = 0, count = 0;
6aa142
+	char *d, *s;
6aa142
+	int count;
6aa142
 
6aa142
 	if (!path || !target || len == 0) {
6aa142
 		errno = EINVAL;
6aa142
 		return -1;
6aa142
 	}
6aa142
 
6aa142
-	memset(devdir, 0, SYSFS_PATH_MAX);
6aa142
-	memset(linkpath, 0, SYSFS_PATH_MAX);
6aa142
-	memset(temp_path, 0, SYSFS_PATH_MAX);
6aa142
-	safestrcpy(devdir, path);
6aa142
-
6aa142
-	if ((readlink(path, linkpath, SYSFS_PATH_MAX)) < 0) {
6aa142
+	count = readlink(path, linkpath, SYSFS_PATH_MAX);
6aa142
+	if (count < 0)
6aa142
 		return -1;
6aa142
-	}
6aa142
-	d = linkpath;
6aa142
+	else
6aa142
+		linkpath[count] = '\0';
6aa142
 	/*
6aa142
 	 * Three cases here:
6aa142
 	 * 1. relative path => format ../..
6aa142
 	 * 2. absolute path => format /abcd/efgh
6aa142
 	 * 3. relative path _from_ this dir => format abcd/efgh
6aa142
 	 */
6aa142
-	switch (*d) {
6aa142
-		case '.':
6aa142
+	if (*linkpath == '/') {
6aa142
+		/* absolute path - copy as is */
6aa142
+		safestrcpymax(target, linkpath, len);
6aa142
+		return 0;
6aa142
+	}
6aa142
+
6aa142
+	safestrcpy(devdir, path);
6aa142
+	s = strrchr(devdir, '/');
6aa142
+	if (s == NULL)
6aa142
+		s = devdir - 1;
6aa142
+	d = linkpath;
6aa142
+	while (*d == '.') {
6aa142
+		if (*(d+1) == '/') {
6aa142
 			/*
6aa142
 			 * handle the case where link is of type ./abcd/xxx
6aa142
 			 */
6aa142
-			safestrcpy(temp_path, devdir);
6aa142
-			if (*(d+1) == '/')
6aa142
-				d += 2;
6aa142
-			else if (*(d+1) == '.')
6aa142
-				goto parse_path;
6aa142
-			s = strrchr(temp_path, '/');
6aa142
-			if (s != NULL) {
6aa142
-				*(s+1) = '\0';
6aa142
-				safestrcat(temp_path, d);
6aa142
-			} else {
6aa142
-				safestrcpy(temp_path, d);
6aa142
-			}
6aa142
-			safestrcpymax(target, temp_path, len);
6aa142
-			break;
6aa142
+			d += 2;
6aa142
+			while (*d == '/')
6aa142
+				d++;
6aa142
+			continue;
6aa142
+		} else if (*(d+1) != '.' || *(d+2) != '/')
6aa142
 			/*
6aa142
-			 * relative path, getting rid of leading "../.."
6aa142
+			 * relative path from this directory, starting
6aa142
+			 * with a hidden directory
6aa142
 			 */
6aa142
-parse_path:
6aa142
-			while (*d == '/' || *d == '.') {
6aa142
-				if (*d == '/')
6aa142
-					slashes++;
6aa142
-				d++;
6aa142
-			}
6aa142
-			d--;
6aa142
-			s = &devdir[strlen(devdir)-1];
6aa142
-			while (s != NULL && count != (slashes+1)) {
6aa142
+			break;
6aa142
+
6aa142
+		/*
6aa142
+		 * relative path, getting rid of leading "../.."; must
6aa142
+		 * be careful here since any path component of devdir
6aa142
+		 * could be a symlink again
6aa142
+		 */
6aa142
+		for (;;) {
6aa142
+			while (s > devdir && *s == '/') {
6aa142
 				s--;
6aa142
-				if (*s == '/')
6aa142
-					count++;
6aa142
+				if (*s == '.'
6aa142
+				    && (s == devdir || *(s-1) == '/'))
6aa142
+					s--;
6aa142
 			}
6aa142
-			safestrcpymax(s, d, (SYSFS_PATH_MAX-strlen(devdir)));
6aa142
-			safestrcpymax(target, devdir, len);
6aa142
-			break;
6aa142
-		case '/':
6aa142
-			/* absolute path - copy as is */
6aa142
-			safestrcpymax(target, linkpath, len);
6aa142
-			break;
6aa142
-		default:
6aa142
-			/* relative path from this directory */
6aa142
-			safestrcpy(temp_path, devdir);
6aa142
-			s = strrchr(temp_path, '/');
6aa142
-			if (s != NULL) {
6aa142
-				*(s+1) = '\0';
6aa142
-				safestrcat(temp_path, linkpath);
6aa142
-			} else {
6aa142
-				safestrcpy(temp_path, linkpath);
6aa142
+			*(s+1) = '\0';
6aa142
+			if (*devdir == '\0' || sysfs_path_is_link(devdir))
6aa142
+				/*
6aa142
+				 * condition will be true eventually
6aa142
+				 * because we already know that all
6aa142
+				 * but the last component of path
6aa142
+				 * resolve to a directory
6aa142
+				 */
6aa142
+				break;
6aa142
+			if (sysfs_get_link(devdir, devdir, SYSFS_PATH_MAX))
6aa142
+				return -1;
6aa142
+			s = devdir + strlen(devdir) - 1;
6aa142
+		}
6aa142
+		while (s >= devdir) {
6aa142
+			if (*s == '/') {
6aa142
+				if (*(s+1) != '.' || *(s+2) != '.'
6aa142
+				    || *(s+3) != '\0') {
6aa142
+					d += 3;
6aa142
+					while (*d == '/')
6aa142
+						d++;
6aa142
+				} else
6aa142
+					s += 2;
6aa142
+				break;
6aa142
 			}
6aa142
-			safestrcpymax(target, temp_path, len);
6aa142
+			s--;
6aa142
+		}
6aa142
+		if (s < devdir || *(s+1) == '\0')
6aa142
+			break;
6aa142
 	}
6aa142
+
6aa142
+	/*
6aa142
+	 * appending to devdir a slash and the (possibly shortened)
6aa142
+	 * relative path to the link source
6aa142
+	 */
6aa142
+	s++;
6aa142
+	if (s > devdir && *s == '\0')
6aa142
+		*s++ = '/';
6aa142
+	*s = '\0';
6aa142
+	safestrcpymax(s, d, SYSFS_PATH_MAX-(s-devdir));
6aa142
+	safestrcpymax(target, devdir, len);
6aa142
 	return 0;
6aa142
 }
6aa142