Blame SOURCES/autofs-5.1.7-cater-for-empty-mounts-list-in-mnts_get_expire_list.patch

beb904
autofs-5.1.7 - cater for empty mounts list in mnts_get_expire_list()
beb904
beb904
From: Ian Kent <raven@themaw.net>
beb904
beb904
Coverity: var_deref_model: Passing null pointer "tree" to
beb904
	  "tree_traverse_inorder", which dereferences it.
beb904
beb904
This obviously can't happen but deal with it anyway to quiet Coverity.
beb904
beb904
Signed-off-by: Ian Kent <raven@themaw.net>
beb904
---
beb904
 CHANGELOG    |    1 +
beb904
 lib/mounts.c |    6 ++++--
beb904
 2 files changed, 5 insertions(+), 2 deletions(-)
beb904
beb904
--- autofs-5.1.4.orig/CHANGELOG
beb904
+++ autofs-5.1.4/CHANGELOG
beb904
@@ -64,6 +64,7 @@
beb904
 - fix missing lock release in mount_subtree().
beb904
 - fix double free in parse_mapent().
beb904
 - refactor lookup_prune_one_cache() a bit.
beb904
+- cater for empty mounts list in mnts_get_expire_list().
beb904
 
beb904
 xx/xx/2018 autofs-5.1.5
beb904
 - fix flag file permission.
beb904
--- autofs-5.1.4.orig/lib/mounts.c
beb904
+++ autofs-5.1.4/lib/mounts.c
beb904
@@ -1445,8 +1445,10 @@ void mnts_get_expire_list(struct list_he
beb904
 		}
beb904
 	}
beb904
 
beb904
-	tree_traverse_inorder(tree, tree_mnt_expire_list_work, mnts);
beb904
-	tree_free(tree);
beb904
+	if (tree) {
beb904
+		tree_traverse_inorder(tree, tree_mnt_expire_list_work, mnts);
beb904
+		tree_free(tree);
beb904
+	}
beb904
 done:
beb904
 	mnts_hash_mutex_unlock();
beb904
 }