Blame SOURCES/autofs-5.1.7-fix-return-from-umount_subtree_mounts-on-offset-list-delete.patch

9a499a
autofs-5.1.7 - fix return from umount_subtree_mounts() on offset list delete
9a499a
9a499a
From: Ian Kent <raven@themaw.net>
9a499a
9a499a
When there are no mounts left in a subtree of offset mounts the offset
9a499a
list is deleted. If all goes well deleting the list this shouldn't cause
9a499a
a positive return from umount_subtree_mounts() (essentially saying not
9a499a
the umount of the subtree has not succeeded).
9a499a
9a499a
Signed-off-by: Ian Kent <raven@themaw.net>
9a499a
---
9a499a
 CHANGELOG          |    1 +
9a499a
 daemon/automount.c |    5 +++--
9a499a
 2 files changed, 4 insertions(+), 2 deletions(-)
9a499a
9a499a
--- autofs-5.1.4.orig/CHANGELOG
9a499a
+++ autofs-5.1.4/CHANGELOG
9a499a
@@ -15,6 +15,7 @@
9a499a
 - simplify mount_subtree() mount check.
9a499a
 - fix mnts_get_expire_list() expire list construction.
9a499a
 - fix inconsistent locking in umount_subtree_mounts().
9a499a
+- fix return from umount_subtree_mounts() on offset list delete.
9a499a
 
9a499a
 xx/xx/2018 autofs-5.1.5
9a499a
 - fix flag file permission.
9a499a
--- autofs-5.1.4.orig/daemon/automount.c
9a499a
+++ autofs-5.1.4/daemon/automount.c
9a499a
@@ -590,9 +590,10 @@ static int umount_subtree_mounts(struct
9a499a
 
9a499a
 		if (!left && is_mm_root) {
9a499a
 			status = cache_delete_offset_list(mc, me->key);
9a499a
-			if (status != CHE_OK)
9a499a
+			if (status != CHE_OK) {
9a499a
 				warn(ap->logopt, "couldn't delete offset list");
9a499a
-			left++;
9a499a
+				left++;
9a499a
+			}
9a499a
 		}
9a499a
 
9a499a
 		if (ap->entry->maps &&