aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIan Kent <raven@themaw.net>2021-04-09 12:51:52 +0800
committerIan Kent <raven@themaw.net>2021-04-12 13:19:27 +0800
commit9d5a5eec124224b708fa774ac279a076c5104ba3 (patch)
treeaff2944c63586dd67766ce15a1333a98c1c4d24f
parentb0fc91cf27f27a666840bcc3619ee8a0d62672ef (diff)
downloadautofs-9d5a5eec124224b708fa774ac279a076c5104ba3.tar.gz
autofs-5.1.7 - fix missing lock release in mount_subtree()
Covarity: missing_unlock: Returning without unlocking "mc->rwlock". Signed-off-by: Ian Kent <raven@themaw.net>
-rw-r--r--CHANGELOG1
-rw-r--r--modules/parse_sun.c1
2 files changed, 2 insertions, 0 deletions
diff --git a/CHANGELOG b/CHANGELOG
index 1d56c96f..ff3d88eb 100644
--- a/CHANGELOG
+++ b/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().
25/01/2021 autofs-5.1.7
- make bind mounts propagation slave by default.
diff --git a/modules/parse_sun.c b/modules/parse_sun.c
index 05f53fc2..5d15f892 100644
--- a/modules/parse_sun.c
+++ b/modules/parse_sun.c
@@ -1105,6 +1105,7 @@ static int mount_subtree(struct autofs_point *ap, struct mapent_cache *mc,
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] = '/';