[LTP] [PATCH v3] Add rename15 test

[Li Wang]

On Thu, Aug 1, 2024 at 9:50 PM Andrea Cervesato <andrea.cervesato@suse.de>
wrote:

> From: Andrea Cervesato <andrea.cervesato@suse.com>
>

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


> This test has been extracted from symlink01 and it verifies that
> rename() is working correctly on symlink() generated files, renaming
> symbolic link and checking if stat() information are preserved.
>
> Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
> ---
> - tst_tmpdir_path/tst_tmpdir_mkpath usage
> - use absolute paths
> - add all_filesystems = 1
>
>  runtest/syscalls                            |   2 +-
>  testcases/kernel/syscalls/rename/.gitignore |   1 +
>  testcases/kernel/syscalls/rename/rename15.c | 109 ++++++++++++++++++++
>  3 files changed, 111 insertions(+), 1 deletion(-)
>  create mode 100644 testcases/kernel/syscalls/rename/rename15.c
>
> diff --git a/runtest/syscalls b/runtest/syscalls
> index 9b3cba667..6028c57c3 100644
> --- a/runtest/syscalls
> +++ b/runtest/syscalls
> @@ -1173,7 +1173,6 @@ removexattr01 removexattr01
>  removexattr02 removexattr02
>
>  rename01 rename01
> -rename01A symlink01 -T rename01
>  rename03 rename03
>  rename04 rename04
>  rename05 rename05
> @@ -1186,6 +1185,7 @@ rename11 rename11
>  rename12 rename12
>  rename13 rename13
>  rename14 rename14
> +rename15 rename15
>
>  #renameat test cases
>  renameat01 renameat01
> diff --git a/testcases/kernel/syscalls/rename/.gitignore
> b/testcases/kernel/syscalls/rename/.gitignore
> index f95cf7d21..d17b80f09 100644
> --- a/testcases/kernel/syscalls/rename/.gitignore
> +++ b/testcases/kernel/syscalls/rename/.gitignore
> @@ -11,3 +11,4 @@
>  /rename12
>  /rename13
>  /rename14
> +/rename15
> diff --git a/testcases/kernel/syscalls/rename/rename15.c
> b/testcases/kernel/syscalls/rename/rename15.c
> new file mode 100644
> index 000000000..234632879
> --- /dev/null
> +++ b/testcases/kernel/syscalls/rename/rename15.c
> @@ -0,0 +1,109 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +/*
> + * Copyright (c) 2000 Silicon Graphics, Inc.  All Rights Reserved.
> + *    Authors: David Fenner, Jon Hendrickson
> + * Copyright (C) 2024 Andrea Cervesato andrea.cervesato@suse.com
> + */
> +
> +/*\
> + * [Description]
> + *
> + * This test verifies that rename() is working correctly on symlink()
> + * generated files.
> + */
> +
> +#include "tst_test.h"
> +#include "tst_tmpdir.h"
> +
> +#define MNTPOINT "mnt"
> +#define OLDNAME MNTPOINT"/msymlink0"
> +#define NEWNAME MNTPOINT"/asymlink0"
> +#define OBJNAME MNTPOINT"/object"
> +
> +static char *tmpdir;
> +static char *oldname;
> +static char *newname;
> +static char *objname;
> +
> +static void test_existing(void)
> +{
> +       tst_res(TINFO, "Test rename() on symlink pointing to an existent
> path");
> +
> +       struct stat oldsym_stat;
> +       struct stat newsym_stat;
> +
> +       SAFE_SYMLINK(tmpdir, oldname);
> +       SAFE_STAT(oldname, &oldsym_stat);
> +
> +       SAFE_RENAME(oldname, newname);
> +       SAFE_STAT(newname, &newsym_stat);
> +
> +       TST_EXP_EQ_LI(oldsym_stat.st_ino, newsym_stat.st_ino);
> +       TST_EXP_EQ_LI(oldsym_stat.st_dev, newsym_stat.st_dev);
> +
> +       SAFE_UNLINK(newname);
> +}
> +
> +static void test_non_existing(void)
> +{
> +       tst_res(TINFO, "Test rename() on symlink pointing to a
> non-existent path");
> +
> +       struct stat path_stat;
> +
> +       SAFE_SYMLINK("this_path_doesnt_exist", oldname);
> +       TST_EXP_FAIL(stat(oldname, &path_stat), ENOENT);
> +
> +       SAFE_RENAME(oldname, newname);
> +       TST_EXP_FAIL(stat(newname, &path_stat), ENOENT);
> +
> +       SAFE_UNLINK(newname);
> +}
> +
> +static void test_creat(void)
> +{
> +
> +       tst_res(TINFO, "Test rename() on symlink pointing to a path
> created lately");
> +
> +       struct stat path_stat;
> +
> +       SAFE_SYMLINK(objname, oldname);
> +       TST_EXP_FAIL(stat(oldname, &path_stat), ENOENT);
> +
> +       tst_res(TINFO, "Create object file");
> +
> +       int fd;
> +
> +       fd = SAFE_CREAT(objname, 0700);
> +       if (fd >= 0)
> +               SAFE_CLOSE(fd);
> +
> +       SAFE_RENAME(oldname, newname);
> +       TST_EXP_PASS(stat(newname, &path_stat));
> +
> +       SAFE_UNLINK(objname);
> +       SAFE_UNLINK(newname);
> +}
> +
> +static void run(void)
> +{
> +       test_existing();
> +       test_creat();
> +       test_non_existing();
> +}
> +
> +static void setup(void)
> +{
> +       tmpdir = tst_tmpdir_path();
> +       oldname = tst_tmpdir_mkpath(OLDNAME);
> +       newname = tst_tmpdir_mkpath(NEWNAME);
> +       objname = tst_tmpdir_mkpath(OBJNAME);
> +}
> +
> +static struct tst_test test = {
> +       .setup = setup,
> +       .test_all = run,
> +       .all_filesystems = 1,
> +       .mntpoint = MNTPOINT,
> +       .format_device = 1,
> +       .needs_root = 1,
> +};
> --
> 2.43.0
>
>
> --
> Mailing list info: https://lists.linux.it/listinfo/ltp
>
>

-- 
Regards,
Li Wang