Re: [PATCH 0/2] Testsuite adjustment and relevant fix

2023-07-19 Thread Corinna Vinschen
On Jul 19 13:41, Jon Turney wrote: > [1/2] has the side effect of flipping test stat06 from working to failing. > [2/2] fixes that > > When run with TDIRECTORY set, libltp just uses that directory and assumes > something else will clean it up. > > When TDIRECTORY is not set, libltp creates a

[PATCH 2/2] Cygwin: Fix Windows file handle leak in stat("file", -1)

2023-07-19 Thread Jon Turney
Don't leak a Windows file handle if stat() is called with a valid filename, but invalid stat buffer pointer. We do not destroy fh if an exception happens in the __try block, which closes a Windows handle it has opened. Fixes: 73151c54d581 ("syscalls.cc (stat_worker): Don't call build_fh_pc with

[PATCH 0/2] Testsuite adjustment and relevant fix

2023-07-19 Thread Jon Turney
[1/2] has the side effect of flipping test stat06 from working to failing. [2/2] fixes that When run with TDIRECTORY set, libltp just uses that directory and assumes something else will clean it up. When TDIRECTORY is not set, libltp creates a subdirectory under /tmp, and when the test is

[PATCH 1/2] Cygwin: testsuite: Drop setting TDIRECTORY

2023-07-19 Thread Jon Turney
Drop setting TDIRECTORY, just use /tmp in the 'test installation' now that we have it. This effectively reverts f3ed5f2fe029d74372aca68b18936e164ff47cf7 Signed-off-by: Jon Turney --- winsup/testsuite/Makefile.am | 8 winsup/testsuite/cygrun.c | 5 +