cdown / rpms / util-linux

Forked from rpms/util-linux 2 years ago
Clone
6717ab
diff -up util-linux-2.23.2/libmount/src/tab.c.kzak util-linux-2.23.2/libmount/src/tab.c
6717ab
--- util-linux-2.23.2/libmount/src/tab.c.kzak	2013-07-30 10:39:26.218738358 +0200
6717ab
+++ util-linux-2.23.2/libmount/src/tab.c	2014-09-25 10:53:43.525269554 +0200
6717ab
@@ -47,6 +47,8 @@
6717ab
 #include "strutils.h"
6717ab
 #include "loopdev.h"
6717ab
 
6717ab
+static int is_mountinfo(struct libmnt_table *tb);
6717ab
+
6717ab
 /**
6717ab
  * mnt_new_table:
6717ab
  *
6717ab
@@ -233,7 +235,7 @@ int mnt_table_get_root_fs(struct libmnt_
6717ab
 	assert(tb);
6717ab
 	assert(root);
6717ab
 
6717ab
-	if (!tb || !root)
6717ab
+	if (!tb || !root || !is_mountinfo(tb))
6717ab
 		return -EINVAL;
6717ab
 
6717ab
 	DBG(TAB, mnt_debug_h(tb, "lookup root fs"));
6717ab
@@ -241,8 +243,6 @@ int mnt_table_get_root_fs(struct libmnt_
6717ab
 	mnt_reset_iter(&itr, MNT_ITER_FORWARD);
6717ab
 	while(mnt_table_next_fs(tb, &itr, &fs) == 0) {
6717ab
 		int id = mnt_fs_get_parent_id(fs);
6717ab
-		if (!id)
6717ab
-			break;		/* @tab is not mountinfo file? */
6717ab
 
6717ab
 		if (!*root || id < root_id) {
6717ab
 			*root = fs;
6717ab
@@ -250,7 +250,7 @@ int mnt_table_get_root_fs(struct libmnt_
6717ab
 		}
6717ab
 	}
6717ab
 
6717ab
-	return root_id ? 0 : -EINVAL;
6717ab
+	return *root ? 0 : -EINVAL;
6717ab
 }
6717ab
 
6717ab
 /**
6717ab
@@ -271,15 +271,13 @@ int mnt_table_next_child_fs(struct libmn
6717ab
 	struct libmnt_fs *fs;
6717ab
 	int parent_id, lastchld_id = 0, chld_id = 0;
6717ab
 
6717ab
-	if (!tb || !itr || !parent)
6717ab
+	if (!tb || !itr || !parent || !is_mountinfo(tb))
6717ab
 		return -EINVAL;
6717ab
 
6717ab
 	DBG(TAB, mnt_debug_h(tb, "lookup next child of '%s'",
6717ab
 				mnt_fs_get_target(parent)));
6717ab
 
6717ab
 	parent_id = mnt_fs_get_id(parent);
6717ab
-	if (!parent_id)
6717ab
-		return -EINVAL;
6717ab
 
6717ab
 	/* get ID of the previously returned child */
6717ab
 	if (itr->head && itr->p != itr->head) {
6717ab
@@ -310,7 +308,7 @@ int mnt_table_next_child_fs(struct libmn
6717ab
 		}
6717ab
 	}
6717ab
 
6717ab
-	if (!chld_id)
6717ab
+	if (!*chld)
6717ab
 		return 1;	/* end of iterator */
6717ab
 
6717ab
 	/* set the iterator to the @chld for the next call */
6717ab
diff -up util-linux-2.23.2/misc-utils/findmnt.c.kzak util-linux-2.23.2/misc-utils/findmnt.c
6717ab
--- util-linux-2.23.2/misc-utils/findmnt.c.kzak	2013-07-30 11:07:35.138395654 +0200
6717ab
+++ util-linux-2.23.2/misc-utils/findmnt.c	2014-09-25 10:49:50.953050560 +0200
6717ab
@@ -826,8 +826,9 @@ static int tab_is_tree(struct libmnt_tab
6717ab
 	if (!itr)
6717ab
 		return 0;
6717ab
 
6717ab
-	if (mnt_table_next_fs(tb, itr, &fs) == 0)
6717ab
-		rc = mnt_fs_get_id(fs) > 0 && mnt_fs_get_parent_id(fs) > 0;
6717ab
+	rc = (mnt_table_next_fs(tb, itr, &fs) == 0 &&
6717ab
+	      mnt_fs_is_kernel(fs) &&
6717ab
+	      mnt_fs_get_root(fs));
6717ab
 
6717ab
 	mnt_free_iter(itr);
6717ab
 	return rc;