autofs-5.1.7 - fix flags check in umount_multi() From: Ian Kent Coverity: operator_confusion: "ap->flags | 1" is always 1/true regardless of the values of its operand. Signed-off-by: Ian Kent --- CHANGELOG | 1 + daemon/automount.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) --- autofs-5.1.4.orig/CHANGELOG +++ autofs-5.1.4/CHANGELOG @@ -56,6 +56,7 @@ - fix possible memory leak in mnts_add_amdmount(). - fix double unlock in parse_mount(). - add length check in umount_subtree_mounts(). +- fix flags check in umount_multi(). xx/xx/2018 autofs-5.1.5 - fix flag file permission. --- autofs-5.1.4.orig/daemon/automount.c +++ autofs-5.1.4/daemon/automount.c @@ -661,7 +661,7 @@ int umount_multi(struct autofs_point *ap /* Check if the autofs mount has browse mode enabled. * If so re-create the directory entry. */ - if (ap->flags | MOUNT_FLAG_GHOST) { + if (ap->flags & MOUNT_FLAG_GHOST) { int ret; /* If the browse directory create fails log an