autofs-5.1.7 - remove redundant assignment in master_add_amd_mount_section_mounts() From: Ian Kent Coverity: missing_lock: Accessing "entry->current" without holding lock "master_mapent.current_mutex". This is initialization not clearing current source. But the field has already been initialized in the master_new_mapent() call. Signed-off-by: Ian Kent --- CHANGELOG | 1 + daemon/master.c | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGELOG b/CHANGELOG index c7bc0c39..f95b1aa6 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -58,6 +58,7 @@ - add length check in umount_subtree_mounts(). - fix flags check in umount_multi(). - dont try umount after stat() ENOENT fail. +- remove redundant assignment in master_add_amd_mount_section_mounts(). 25/01/2021 autofs-5.1.7 - make bind mounts propagation slave by default. diff --git a/daemon/master.c b/daemon/master.c index 30d7cf98..84743f80 100644 --- a/daemon/master.c +++ b/daemon/master.c @@ -996,7 +996,6 @@ static void master_add_amd_mount_section_mounts(struct master *master, time_t ag source->master_line = 0; entry->age = age; - entry->current = NULL; master_add_mapent(master, entry); next: