[PATCH 0/2] Testsuite adjustment and relevant fix
Corinna Vinschen
corinna-cygwin@cygwin.com
Wed Jul 19 15:33:43 GMT 2023
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 subdirectory under /tmp, and when
> the test is completed, removes the expected files and verifies that the
> directory is empty.
>
> stat06 fails that check, because it creates the a file named "file" there, and
> tries stat("file", -1), testing that it returns the expected value EFAULT.
>
> "file" is removed, but lingers in the STATUS_DELETE_PENDING state until the
> Windows handle which stat_worker() leaks when an exception occurs is closed
> (when the processes exits).
Great find. Please push.
> Future work: It looks like similar problems might generically occur in similar
> code througout syscalls.cc.
Uh oh...
Corinna
More information about the Cygwin-patches
mailing list