Blob Blame History Raw
autofs-5.1.7 - fix missing lock release in mount_subtree()

From: Ian Kent <raven@themaw.net>

Covarity: missing_unlock: Returning without unlocking "mc->rwlock".

Signed-off-by: Ian Kent <raven@themaw.net>
---
 CHANGELOG           |    1 +
 modules/parse_sun.c |    1 +
 2 files changed, 2 insertions(+)

--- autofs-5.1.4.orig/CHANGELOG
+++ autofs-5.1.4/CHANGELOG
@@ -61,6 +61,7 @@
 - remove redundant assignment in master_add_amd_mount_section_mounts().
 - fix dead code in mnts_add_mount().
 - fix arg not used in error print.
+- fix missing lock release in mount_subtree().
 
 xx/xx/2018 autofs-5.1.5
 - fix flag file permission.
--- autofs-5.1.4.orig/modules/parse_sun.c
+++ autofs-5.1.4/modules/parse_sun.c
@@ -1107,6 +1107,7 @@ static int mount_subtree(struct autofs_p
 		len = mount_fullpath(key, PATH_MAX, ap->path, ap->len, me->key);
 		if (!len) {
 			warn(ap->logopt, "path loo long");
+			cache_unlock(mc);
 			return 1;
 		}
 		key[len] = '/';