[ruby-dev:28930] Re: IO.popen("echo a") clear O_APPEND of other fds.

なかだです。

At Sun, 2 Jul 2006 23:13:48 +0900,
Tanaka Akira wrote in [ruby-dev:28924]:
> 以下のように、IO.popen("echo a") が関係ない他の IO の
> O_APPEND を消してしまうようです。

3以上のfdにCLOEXECフラグをセットするときに、元のフラグを見てい
ませんでした。もしかしたら単にcloseするだけのほうがいいのかもし
れません。


Index: io.c
===================================================================
RCS file: /cvs/ruby/src/ruby/io.c,v
retrieving revision 1.411
diff -p -U 2 -r1.411 io.c
--- io.c	2 Jul 2006 02:25:11 -0000	1.411
+++ io.c	2 Jul 2006 16:40:22 -0000
@@ -2952,5 +2952,7 @@ popen_exec(void *pp)
     for (fd = 3; fd < NOFILE; fd++) {
 #ifdef FD_CLOEXEC
-	fcntl(fd, F_SETFL, FD_CLOEXEC);
+	int flag = fcntl(fd, F_GETFL);
+	if ((flag == -1) || (flag & FD_CLOEXEC)) continue;
+	fcntl(fd, F_SETFL, flag | FD_CLOEXEC);
 #else
 	close(fd);


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦