[LTP] [PATCH 1/1] fsmount01: Avoid leaving file descriptor open

Viresh Kumar viresh.kumar@linaro.org
Wed Feb 19 09:33:10 CET 2020


On 19-02-20, 09:30, Petr Vorel wrote:
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
> Hi,
> 
> this is probably not needed, as fd will be closed at program exit
> anyway.
> 
> Kind regards,
> Petr
> 
>  testcases/kernel/syscalls/fsmount/fsmount01.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/testcases/kernel/syscalls/fsmount/fsmount01.c b/testcases/kernel/syscalls/fsmount/fsmount01.c
> index 83185b48a..aea050506 100644
> --- a/testcases/kernel/syscalls/fsmount/fsmount01.c
> +++ b/testcases/kernel/syscalls/fsmount/fsmount01.c
> @@ -16,6 +16,7 @@
>  
>  #define LINELENGTH 256
>  #define MNTPOINT "newmount_point"
> +
>  static int sfd, mfd, is_mounted;
>  
>  static int ismount(char *mntpoint)
> @@ -38,6 +39,9 @@ static int ismount(char *mntpoint)
>  
>  static void cleanup(void)
>  {
> +	if (sfd > 0)
> +		SAFE_CLOSE(sfd);
> +
>  	if (is_mounted)
>  		SAFE_UMOUNT(MNTPOINT);
>  }
> @@ -67,7 +71,6 @@ static void test_fsmount(void)
>  		tst_brk(TBROK | TTERRNO, "fsmount() failed to create a mount object");
>  	mfd = TST_RET;
>  	tst_res(TPASS, "fsmount() created a mount object");
> -	SAFE_CLOSE(sfd);
>  
>  	TEST(move_mount(mfd, "", AT_FDCWD, MNTPOINT, MOVE_MOUNT_F_EMPTY_PATH));
>  	if (TST_RET < 0)

My patchset is making enough changes to this file and fixes this issue as well.
I won't ask you to not get this merged as you already sent it before me :)

My V2 will be on list very soon.

-- 
viresh


More information about the ltp mailing list