diff options
| author | Ian Kent <raven@themaw.net> | 2021-04-09 12:08:09 +0800 |
|---|---|---|
| committer | Ian Kent <raven@themaw.net> | 2021-04-12 13:17:45 +0800 |
| commit | e248b7bea99226dcdfd42572928a154674cdb8fa (patch) | |
| tree | 77194bf6910437ae23eb9ea8079760e256af01cf | |
| parent | 93fb75a5b44b3488b0047eabec3e0b210ae76791 (diff) | |
| download | autofs-e248b7bea99226dcdfd42572928a154674cdb8fa.tar.gz | |
autofs-5.1.7 - remove redundant assignment in master_add_amd_mount_section_mounts()
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 <raven@themaw.net>
| -rw-r--r-- | CHANGELOG | 1 | ||||
| -rw-r--r-- | daemon/master.c | 1 |
2 files changed, 1 insertions, 1 deletions
@@ -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: |