From 452bf3b9c954d770ccb704cb9f5470f515e00584 Mon Sep 17 00:00:00 2001 From: naruse Date: Tue, 8 Nov 2011 08:26:00 +0000 Subject: * ext/pty/pty.c (get_device_once): FreeBSD 8 supported O_CLOEXEC flag for posix_openpt, but FreeBSD 9's posix_openpt doesn't support O_CLOEXEC and fails if specified. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@33668 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 5 +++-- ext/pty/pty.c | 8 ++++---- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 86d686ea02..c2b4683e47 100644 --- a/ChangeLog +++ b/ChangeLog @@ -6,8 +6,9 @@ Tue Nov 8 14:50:55 2011 NAKAMURA Usaku Tue Nov 8 11:01:04 2011 NARUSE, Yui - * ext/pty/pty.c (get_device_once): FreeBSD's posix_openpt doesn't - support O_CLOEXEC and fails if specified. + * ext/pty/pty.c (get_device_once): FreeBSD 8 supported O_CLOEXEC flag + for posix_openpt, but FreeBSD 9's posix_openpt doesn't support + O_CLOEXEC and fails if specified. Tue Nov 8 02:36:45 2011 NAKAMURA Usaku diff --git a/ext/pty/pty.c b/ext/pty/pty.c index fafc2c55dd..9527004f3a 100644 --- a/ext/pty/pty.c +++ b/ext/pty/pty.c @@ -291,19 +291,19 @@ get_device_once(int *master, int *slave, char SlaveName[DEVICELEN], int nomesg, dfl.sa_flags = 0; sigemptyset(&dfl.sa_mask); -#if defined(__sun) +#if defined(__sun) || defined(__FreeBSD__) /* workaround for Solaris 10: grantpt() doesn't work if FD_CLOEXEC is set. [ruby-dev:44688] */ + /* FreeBSD 8 supported O_CLOEXEC for posix_openpt, but FreeBSD 9 removed it */ if ((masterfd = posix_openpt(O_RDWR|O_NOCTTY)) == -1) goto error; if (sigaction(SIGCHLD, &dfl, &old) == -1) goto error; if (grantpt(masterfd) == -1) goto grantpt_error; rb_fd_fix_cloexec(masterfd); #else flags = O_RDWR|O_NOCTTY; -# if defined(O_CLOEXEC) && !defined(__FreeBSD__) +# if defined(O_CLOEXEC) /* glibc posix_openpt() in GNU/Linux calls open("/dev/ptmx", flags) internally. * So version dependency on GNU/Linux is same as O_CLOEXEC with open(). - * O_CLOEXEC is available since Linux 2.6.23. Linux 2.6.18 silently ignore it. - * FreeBSD's posix_openpt doesn't support O_CLOEXEC and fails if specified*/ + * O_CLOEXEC is available since Linux 2.6.23. Linux 2.6.18 silently ignore it. */ flags |= O_CLOEXEC; # endif if ((masterfd = posix_openpt(flags)) == -1) goto error; -- cgit v1.2.3