[LTP] [PATCH v2 2/2] mount: check path exists before tst_is_mounted in cleanup

[Li Wang via ltp <ltp@lists.linux.it>]

tst_is_mounted() now internally resolves the mount path via realpath(),
which will fail if the path does not exist. In sometest (e.g. mount05),
MNTPOINT2 may not exist yet during cleanup, calling tst_is_mounted()
directly could produce misleading warnings or errors.

Add an explicit access(PATH, F_OK) check before calling tst_is_mounted()
in the cleanup, ensuring we only query mounts for existing paths.

Signed-off-by: Li Wang <liwang@redhat.com>
---

Notes:
    v1 --> v2:
            patch 1/2, keep no change, do not send v2.
            patch 2/2, adding two more files for path exist check

 testcases/kernel/containers/mqns/mqns_03.c | 2 +-
 testcases/kernel/containers/mqns/mqns_04.c | 2 +-
 testcases/kernel/syscalls/mount/mount05.c  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/testcases/kernel/containers/mqns/mqns_03.c b/testcases/kernel/containers/mqns/mqns_03.c
index 4d3bfc52f..ac8311559 100644
--- a/testcases/kernel/containers/mqns/mqns_03.c
+++ b/testcases/kernel/containers/mqns/mqns_03.c
@@ -130,7 +130,7 @@ static void cleanup(void)
 	if (!access(MQUEUE2, F_OK))
 		SAFE_MQ_UNLINK(MQNAME2);
 
-	if (tst_is_mounted(DEVDIR))
+	if (!access(DEVDIR, F_OK) && tst_is_mounted(DEVDIR))
 		SAFE_UMOUNT(DEVDIR);
 }
 
diff --git a/testcases/kernel/containers/mqns/mqns_04.c b/testcases/kernel/containers/mqns/mqns_04.c
index d28c330c4..790607ecd 100644
--- a/testcases/kernel/containers/mqns/mqns_04.c
+++ b/testcases/kernel/containers/mqns/mqns_04.c
@@ -123,7 +123,7 @@ static void cleanup(void)
 	if (!access(MQUEUE2, F_OK))
 		SAFE_MQ_UNLINK(MQNAME2);
 
-	if (tst_is_mounted(DEVDIR))
+	if (!access(DEVDIR, F_OK) && tst_is_mounted(DEVDIR))
 		SAFE_UMOUNT(DEVDIR);
 }
 
diff --git a/testcases/kernel/syscalls/mount/mount05.c b/testcases/kernel/syscalls/mount/mount05.c
index 66e102a32..5585e230e 100644
--- a/testcases/kernel/syscalls/mount/mount05.c
+++ b/testcases/kernel/syscalls/mount/mount05.c
@@ -35,7 +35,7 @@ static void cleanup(void)
 	if (tst_is_mounted(MNTPOINT1))
 		SAFE_UMOUNT(MNTPOINT1);
 
-	if (tst_is_mounted(MNTPOINT2))
+	if (!access(MNTPOINT2, F_OK) && tst_is_mounted(MNTPOINT2))
 		SAFE_UMOUNT(MNTPOINT2);
 }
 
-- 
2.49.0


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp