* thread_pthread.c (native_stop_timer_thread): skip to close

  communication pipe to avoid timing bug (process termination timing).
  The communication pipe will closed by OS.



git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@32276 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 6 insertions, 3 deletions

diff --git a/thread_pthread.c b/thread_pthread.c
index 420a3f4db5..016ee32abb 100644
--- a/thread_pthread.c
+++ b/thread_pthread.c
@@ -1189,10 +1189,13 @@ native_stop_timer_thread(int close_anyway)
 	/* close communication pipe */
 	if (close_anyway) {
 	    /* TODO: Uninstall all signal handlers or mask all signals.
-	     *       This pass is cleaning phase.  It is too rare case
-             *       to generate problem, so we remains it in TODO.
+	     *       This pass is cleaning phase (terminate ruby process).
+	     *       To avoid such race, we skip to close communication
+	     *       pipe.  OS will close it at process termination.
+	     *       It may not good practice, but pragmatic.
+	     *       We remain it is TODO.
 	     */
-	    close_communication_pipe();
+	    /* close_communication_pipe(); */
 	}
     }
     return stopped;