[ruby-list:25561] Re: open3.rb

At Fri, 20 Oct 2000 22:49:06 +0900,
Yukihiro Matsumoto wrote:
> まつもと ゆきひろです
> 
> In message "[ruby-list:25538] open3.rb"
>     on 00/10/20, Takaaki Tateishi <[e-mail]> writes:
> 
> |open3.rbについての修正案です．
> |Open3.popen3では，forkを行なってexecすることで3つの入出力を獲得
> |していますが，waitをかけていないために，waitを忘れると，ゾンビが
> |大量に発生してしまうことがありました．
> 
> そおかあ、1秒後にProcess.waitpidするとかでどうでしょう？

　子がさらに fork して孫を作ってすぐに死んでやると、親はすぐに
子の亡骸を wait で回収でき、孫は死んだ時点で orphan となるので
init に回収されると思います。

Index: open3.rb
===================================================================
RCS file: /home/ruby/cvs/ruby/lib/open3.rb,v
retrieving revision 1.4
diff -u -r1.4 open3.rb
--- open3.rb	2000/10/20 16:37:01	1.4
+++ open3.rb	2000/10/21 09:00:12
@@ -16,29 +16,29 @@
 
     pid = fork{
       # child
-      pw[1].close
-      STDIN.reopen(pw[0])
-      pw[0].close
+      fork{
+	# grandchild
+	pw[1].close
+	STDIN.reopen(pw[0])
+	pw[0].close
 
-      pr[0].close
-      STDOUT.reopen(pr[1])
-      pr[1].close
+	pr[0].close
+	STDOUT.reopen(pr[1])
+	pr[1].close
 
-      pe[0].close
-      STDERR.reopen(pe[1])
-      pe[1].close
+	pe[0].close
+	STDERR.reopen(pe[1])
+	pe[1].close
 
-      exec(cmd)
+	exec(cmd)
+      }
       _exit 127
     }
 
     pw[0].close
     pr[1].close
     pe[1].close
-    Thread.start do
-      sleep 1
-      Process.waitpid(pid)
-    end
+    Process.waitpid(pid)
     pi = [ pw[1], pr[0], pe[0] ]
     if defined? yield
       return yield *pi


-- 
                           /
                          /__  __
                         / )  )  ) )  /    and.or.jp / ruby-lang.org
Akinori -Aki- MUSHA aka / (_ /  ( (__(  @ idaemons.org / FreeBSD.org

"We're only at home when we're on the run, on the wing, on the fly"