Remove the leftover initialisation of left from when there were multiple calls to umount_subtree_mounts(). Signed-off-by: Ian Kent <raven@xxxxxxxxxx> --- CHANGELOG | 1 + daemon/automount.c | 4 +--- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/CHANGELOG b/CHANGELOG index 2d312c9fc..f92d01b07 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -37,6 +37,7 @@ - fix cache writelock must be taken in update_map_cache(). - fix skip valid map entries on expire cleanup. - remove unnecessary call to set_direct_mount_tree_catatonic(). +- remove unnecessary assignment in umount_multi(). 02/11/2023 autofs-5.1.9 - fix kernel mount status notification. diff --git a/daemon/automount.c b/daemon/automount.c index 5578a965b..c2c8d8f00 100644 --- a/daemon/automount.c +++ b/daemon/automount.c @@ -710,9 +710,7 @@ int umount_multi(struct autofs_point *ap, const char *path, int incl) mnts_put_mount(sbmnt); } - left = 0; - - left += umount_subtree_mounts(ap, path, is_autofs_fs); + left = umount_subtree_mounts(ap, path, is_autofs_fs); /* Delete detritus like unwanted mountpoints and symlinks */ if (left == 0 && -- 2.50.1