diff --git a/ChangeLog b/ChangeLog index 77353084b7..ad8363ecb1 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Mon Jun 13 23:28:50 2011 Nobuyoshi Nakada + + * ext/io/console/console.c (console_dev): take care of no-ctty + case. + Mon Jun 13 23:06:12 2011 KOSAKI Motohiro * thread_pthread.c: rewrite GVL completely. diff --git a/ext/io/console/console.c b/ext/io/console/console.c index cc4ab11380..c2660d2ac9 100644 --- a/ext/io/console/console.c +++ b/ext/io/console/console.c @@ -557,27 +557,36 @@ console_dev(VALUE klass) VALUE out; rb_io_t *ofptr; #endif + int fd, mode; - args[1] = INT2FIX(O_RDWR); #ifdef CONSOLE_DEVICE_FOR_WRITING - args[0] = rb_str_new2(CONSOLE_DEVICE_FOR_WRITING); + fd = open(CONSOLE_DEVICE_FOR_WRITING, O_WRONLY); + if (fd < 0) return Qnil; + args[1] = INT2FIX(O_WRONLY); + args[0] = INT2NUM(fd); out = rb_class_new_instance(2, args, klass); #endif - args[0] = rb_str_new2(CONSOLE_DEVICE_FOR_READING); - con = rb_class_new_instance(2, args, klass); + fd = open(CONSOLE_DEVICE_FOR_READING, O_RDWR); + if (fd < 0) { #ifdef CONSOLE_DEVICE_FOR_WRITING + rb_io_close(out); +#endif + return Qnil; + } + args[1] = INT2FIX(O_RDWR); + args[0] = INT2NUM(fd); + con = rb_class_new_instance(2, args, klass); GetOpenFile(con, fptr); + fptr->pathv = rb_obj_freeze(rb_str_new2(CONSOLE_DEVICE)); +#ifdef CONSOLE_DEVICE_FOR_WRITING GetOpenFile(out, ofptr); # ifdef HAVE_RB_IO_GET_WRITE_IO -# ifdef _WIN32 - ofptr->pathv = fptr->pathv = rb_str_new2(CONSOLE_DEVICE); -# endif + ofptr->pathv = fptr->pathv; fptr->tied_io_for_writing = out; # else fptr->f2 = ofptr->f; ofptr->f = 0; # endif - fptr->mode |= FMODE_WRITABLE; #endif rb_const_set(klass, id_console, con); } diff --git a/test/io/console/test_io_console.rb b/test/io/console/test_io_console.rb index 030dcd38ed..5475a93fa3 100644 --- a/test/io/console/test_io_console.rb +++ b/test/io/console/test_io_console.rb @@ -160,3 +160,13 @@ class TestIO_Console < Test::Unit::TestCase s.close if s end end if defined?(PTY) and defined?(IO::console) + +class TestIO_Console < Test::Unit::TestCase + require_relative '../../ruby/envutil' + + def test_noctty + assert_in_out_err(["-rio/console"], + "Process.daemon(true, true); p IO.console", + ["nil"]) + end +end if defined?(Process.daemon) and defined?(IO::console)