diff options
Diffstat (limited to 'nptl/tst-cancel21.c')
-rw-r--r-- | nptl/tst-cancel21.c | 17 |
1 files changed, 10 insertions, 7 deletions
diff --git a/nptl/tst-cancel21.c b/nptl/tst-cancel21.c index b54f236b03..c23faa927f 100644 --- a/nptl/tst-cancel21.c +++ b/nptl/tst-cancel21.c @@ -1,4 +1,4 @@ -/* Copyright (C) 2003-2015 Free Software Foundation, Inc. +/* Copyright (C) 2003-2016 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Jakub Jelinek <jakub@redhat.com>, 2003. @@ -123,12 +123,6 @@ tf (void *arg) exit (1); } - /* This will cause the read in the initial thread to return. */ - close (fd[0]); - close (fd[1]); - close (fd[2]); - close (fd[3]); - void *ret; if (pthread_join (th, &ret) != 0) { @@ -154,6 +148,15 @@ tf (void *arg) exit (1); } + /* The pipe closing must be issued after the cancellation handling to avoid + a race condition where the cancellation runs after both pipe ends are + closed. In this case the read syscall returns EOF and the cancellation + must not act. */ + close (fd[0]); + close (fd[1]); + close (fd[2]); + close (fd[3]); + exit (0); } |