Blame SOURCES/sysfsutils-2.1.0-get_link.patch

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