Bug #4920: Process.daemon()呼び出しによりタイマースレッドが２つ出来てしまう

• Status: Closed
• Assignee: nobu (Nobuyoshi Nakada)
• Target version: 1.9.3
• ruby -v: ruby 1.9.3dev (2011-06-19 trunk 32170) [x86_64-linux]

[kosaki (Motohiro KOSAKI)]

ささださんとakrさんが発見してくださったのですが、現在 trunkで

$ ./ruby -e 'Process.daemon(true,true); p Dir.entries("/proc/self/task")'

と実行すると、スレッドが３匹います(Linuxで実行してください）。何故かというと Process.daemon()
を契機にタイマースレッドが２つ出来てしまうからです。

原因は以下のコードです

---

static VALUE
proc_daemon(int argc, VALUE *argv)
{
VALUE nochdir, noclose;
int n;

rb_secure(2);
rb_scan_args(argc, argv, "02", &nochdir, &noclose);

prefork();
before_fork();
n = daemon(RTEST(nochdir), RTEST(noclose));
after_fork();
if (n < 0) rb_sys_fail("daemon");
return INT2FIX(n);

}

BSD以外の環境ではdaemon()はOSのdaemon(3)ではなく、rb_daemon()を使用し、
rb_daemonは内部的にrb_fork_err()を使用するため結果的に

before_fork();
before_fork();
fork();
after_fork();
after_fork();

という呼び出し順序になり、after_fork()のたびにタイマースレッドをつくるので合計２スレッド。

もちろん、1.9.2ではちゃんと動くのでこれはリグレッションです。
さすがに、これはダメだろうと思うので時期が時期だけに恐縮ですがターゲットを1.9.3に
設定させていただきます。

[kosaki (Motohiro KOSAKI)]

直接の原因はこのコミットのようです。

commit afbd5661a0ca041ac818c9e5732911c5db21c9f9
Author: nobu nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Date: Tue Jul 13 12:31:17 2010 +0000

* process.c (rb_daemon): split from proc_daemon.


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

---

process.c | 27 +++++++++++++++++++--------
1 files changed, 19 insertions(+), 8 deletions(-)

diff --git a/process.c b/process.c
index 8e601ba..84fed60 100644
--- a/process.c
+++ b/process.c
@@ -4554,6 +4554,11 @@ proc_setmaxgroups(VALUE obj, VALUE val)
}

#if defined(HAVE_DAEMON) || (defined(HAVE_FORK) && defined(HAVE_SETSID))
+#ifndef HAVE_DAEMON
+static int rb_daemon(int nochdir, int noclose);
+#define daemon(nochdir, noclose) rb_daemon(nochdir, noclose)
+#endif
+
/*

• call-seq:

• Process.daemon()                        -> 0
  

@@ -4577,14 +4582,20 @@ proc_daemon(int argc, VALUE *argv)
rb_secure(2);
rb_scan_args(argc, argv, "02", &nochdir, &noclose);

-#if defined(HAVE_DAEMON)
prefork();
before_fork();
n = daemon(RTEST(nochdir), RTEST(noclose));
after_fork();
if (n < 0) rb_sys_fail("daemon");
return INT2FIX(n);
-#elif defined(HAVE_FORK)
+}
+
+#ifndef HAVE_DAEMON
+static int
+rb_daemon(int nochdir, int noclose)
+{

• int n, err = 0;
• switch (rb_fork(0, 0, 0, Qnil)) {
  case -1:
  rb_sys_fail("daemon");
  @@ -4599,26 +4610,26 @@ proc_daemon(int argc, VALUE argv)
  / must not be process-leader */
  switch (rb_fork(0, 0, 0, Qnil)) {
  case -1:

•   rb_sys_fail("daemon");
  

•   return -1;
   case 0:
    break;
   default:
    _exit(EXIT_SUCCESS);
  

  }

• if (!RTEST(nochdir))

•   (void)chdir("/");
  

• if (!nochdir)

•   err = chdir("/");
  

• if (!RTEST(noclose) && (n = open("/dev/null", O_RDWR, 0)) != -1) {

• if (!noclose && (n = open("/dev/null", O_RDWR, 0)) != -1) {
  (void)dup2(n, 0);
  (void)dup2(n, 1);
  (void)dup2(n, 2);
  if (n > 2)
  (void)close (n);
  }

• return INT2FIX(0);
  -#endif

• return err;
  }
  +#endif
  #else
  #define proc_daemon rb_f_notimplement
  #endif

[kosaki (Motohiro KOSAKI)]

• Status changed from Assigned to Closed
• Assignee changed from kosaki (Motohiro KOSAKI) to nobu (Nobuyoshi Nakada)

fixed by r32221.