[LTP] [PATCH] syscalls/creat08.c: Do not move temporary directory in loops
Petr Vorel
pvorel@suse.cz
Tue Feb 27 12:07:11 CET 2018
Hi Jinhui,
> During running creat08 in loops, the deletion of temporary
> directory leads to failure. as below:
> ----------------------------------------------------------
> creat08 1 TPASS : Test passed in block0.
> creat08 2 TPASS : Test passed in block1.
> creat08 3 TPASS : Test passed in block2.
> creat08 4 TPASS : Test passed in block3
> creat08 5 TFAIL : creat08.c:143: Creation of testdir.A.24370 failed
> ...
> ----------------------------------------------------------
> Signed-off-by: Jinhui Huang <huangjh.jy@cn.fujitsu.com>
> ---
> testcases/kernel/syscalls/creat/creat08.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
> diff --git a/testcases/kernel/syscalls/creat/creat08.c b/testcases/kernel/syscalls/creat/creat08.c
> index 284ae5f..50f2b39 100644
> --- a/testcases/kernel/syscalls/creat/creat08.c
> +++ b/testcases/kernel/syscalls/creat/creat08.c
> @@ -70,6 +70,7 @@ int local_flag;
> #define ROOT_SETGID "root_setgid"
> #define MSGSIZE 150
> +static void tst_cleanup(void);
> static void cleanup(void);
> static void setup(void);
> @@ -423,8 +424,9 @@ int main(int ac, char **av)
> } else {
> tst_resm(TFAIL, "Test failed in block3");
> }
> - cleanup();
> + tst_cleanup();
> }
> + cleanup();
> tst_exit();
> }
> @@ -434,7 +436,7 @@ static void setup(void)
> tst_tmpdir();
> }
> -static void cleanup(void)
> +static void tst_cleanup(void)
> {
> if (unlink(setgid_A) == -1) {
> tst_resm(TBROK, "%s failed", setgid_A);
> @@ -447,6 +449,9 @@ static void cleanup(void)
> SAFE_UNLINK(NULL, root_setgid_B);
> SAFE_UNLINK(NULL, nosetgid_B);
> SAFE_RMDIR(NULL, DIR_B);
> +}
> +static void cleanup(void)
> +{
> tst_rmdir();
> }
Pushed, thanks!
Kind regards,
Petr
More information about the ltp
mailing list