Ian Kent 346236
autofs-5.0.7 - fix use cache entry after free mistake
Ian Kent 346236
Ian Kent 346236
From: Ian Kent <ikent@redhat.com>
Ian Kent 346236
Ian Kent 346236
Fix an obvious use after free mistake in lookup_prune_one_cache().
Ian Kent 346236
---
Ian Kent 346236
Ian Kent 346236
 CHANGELOG       |    1 +
Ian Kent 346236
 daemon/lookup.c |    7 +++++--
Ian Kent 346236
 2 files changed, 6 insertions(+), 2 deletions(-)
Ian Kent 346236
Ian Kent 346236
Ian Kent 346236
diff --git a/CHANGELOG b/CHANGELOG
Ian Kent 346236
index faf4c80..dc38580 100644
Ian Kent 346236
--- a/CHANGELOG
Ian Kent 346236
+++ b/CHANGELOG
Ian Kent 346236
@@ -1,6 +1,7 @@
Ian Kent 346236
 ??/??/2012 autofs-5.0.8
Ian Kent 346236
 =======================
Ian Kent 346236
 - fix nobind sun escaped map entries.
Ian Kent 346236
+- fix use cache entry after free in lookup_prune_one_cache().
Ian Kent 346236
 
Ian Kent 346236
 25/07/2012 autofs-5.0.7
Ian Kent 346236
 =======================
Ian Kent 346236
diff --git a/daemon/lookup.c b/daemon/lookup.c
Ian Kent 346236
index 7909536..e3d9536 100644
Ian Kent 346236
--- a/daemon/lookup.c
Ian Kent 346236
+++ b/daemon/lookup.c
Ian Kent 346236
@@ -1103,15 +1103,18 @@ void lookup_prune_one_cache(struct autofs_point *ap, struct mapent_cache *mc, ti
Ian Kent 346236
 		if (valid)
Ian Kent 346236
 			cache_delete(mc, key);
Ian Kent 346236
 		else if (!is_mounted(_PROC_MOUNTS, path, MNTS_AUTOFS)) {
Ian Kent 346236
+			dev_t devid = ap->dev;
Ian Kent 346236
 			status = CHE_FAIL;
Ian Kent 346236
+			if (ap->type == LKP_DIRECT)
Ian Kent 346236
+				devid = this->dev;
Ian Kent 346236
 			if (this->ioctlfd == -1)
Ian Kent 346236
 				status = cache_delete(mc, key);
Ian Kent 346236
 			if (status != CHE_FAIL) {
Ian Kent 346236
 				if (ap->type == LKP_INDIRECT) {
Ian Kent 346236
 					if (ap->flags & MOUNT_FLAG_GHOST)
Ian Kent 346236
-						rmdir_path(ap, path, ap->dev);
Ian Kent 346236
+						rmdir_path(ap, path, devid);
Ian Kent 346236
 				} else
Ian Kent 346236
-					rmdir_path(ap, path, this->dev);
Ian Kent 346236
+					rmdir_path(ap, path, devid);
Ian Kent 346236
 			}
Ian Kent 346236
 		}
Ian Kent 346236
 		cache_unlock(mc);