diff options
author | Ian Kent <raven@themaw.net> | 2021-04-09 12:51:52 +0800 |
---|---|---|
committer | Ian Kent <raven@themaw.net> | 2021-04-12 13:19:27 +0800 |
commit | 9d5a5eec124224b708fa774ac279a076c5104ba3 (patch) | |
tree | aff2944c63586dd67766ce15a1333a98c1c4d24f | |
parent | b0fc91cf27f27a666840bcc3619ee8a0d62672ef (diff) | |
download | autofs-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-- | CHANGELOG | 1 | ||||
-rw-r--r-- | modules/parse_sun.c | 1 |
2 files changed, 2 insertions, 0 deletions
@@ -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] = '/'; |