[前][次][番号順一覧][スレッド一覧]

ruby-changes:30929

From: nobu <ko1@a...>
Date: Sat, 21 Sep 2013 20:11:28 +0900 (JST)
Subject: [ruby-changes:30929] nobu:r43008 (trunk): process.c: remove cloexec setting

nobu	2013-09-21 20:11:10 +0900 (Sat, 21 Sep 2013)

  New Revision: 43008

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=43008

  Log:
    process.c: remove cloexec setting
    
    * process.c (rb_fork_internal): remove cloexec setting on pipes
      created by rb_cloexec_pipe.  patch by normalperson (Eric Wong) at
      [ruby-core:56523].  [Bug #8769]

  Modified files:
    trunk/ChangeLog
    trunk/process.c
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 43007)
+++ ChangeLog	(revision 43008)
@@ -1,3 +1,9 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1
+Sat Sep 21 20:11:06 2013  Nobuyoshi Nakada  <nobu@r...>
+
+	* process.c (rb_fork_internal): remove cloexec setting on pipes
+	  created by rb_cloexec_pipe.  patch by normalperson (Eric Wong) at
+	  [ruby-core:56523].  [Bug #8769]
+
 Sat Sep 21 01:04:25 2013  Zachary Scott  <e@z...>
 
 	* lib/benchmark.rb: [DOC] grammar of Benchmark#bm [Bug #8888]
Index: process.c
===================================================================
--- process.c	(revision 43007)
+++ process.c	(revision 43008)
@@ -3389,10 +3389,6 @@ rb_fork_internal(int *status, int (*chfu https://github.com/ruby/ruby/blob/trunk/process.c#L3389
     }
     else {
 	if (pipe_nocrash(ep, fds)) return -1;
-	if (fcntl(ep[1], F_SETFD, FD_CLOEXEC)) {
-	    preserving_errno((close(ep[0]), close(ep[1])));
-	    return -1;
-	}
         pid = retry_fork(status, ep, chfunc_is_async_signal_safe);
         if (pid < 0)
             return pid;

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]