diff options
| author | ksss <[email protected]> | 2017-05-27 13:29:38 +0900 |
|---|---|---|
| committer | ksss <[email protected]> | 2017-05-27 13:34:11 +0900 |
| commit | a3a4f4d061ed4fc1debbee0fe172a786c7c152d5 (patch) | |
| tree | eab3f3fe0b03cef0e9db381fbef3c732b4baa565 | |
| parent | 0c0b34136e17b2452e3122f22813ca71d75434fd (diff) | |
| download | mruby-a3a4f4d061ed4fc1debbee0fe172a786c7c152d5.tar.gz mruby-a3a4f4d061ed4fc1debbee0fe172a786c7c152d5.zip | |
IO#sysread Check for readable
| -rw-r--r-- | src/io.c | 3 | ||||
| -rw-r--r-- | test/io.rb | 5 |
2 files changed, 8 insertions, 0 deletions
@@ -637,6 +637,9 @@ mrb_io_sysread(mrb_state *mrb, mrb_value io) } fptr = (struct mrb_io *)io_get_open_fptr(mrb, io); + if (!fptr->readable) { + mrb_raise(mrb, E_IO_ERROR, "not opened for reading"); + } ret = read(fptr->fd, RSTRING_PTR(buf), maxlen); switch (ret) { case 0: /* EOF */ diff --git a/test/io.rb b/test/io.rb index 9ce8985f7..ee4309b57 100644 --- a/test/io.rb +++ b/test/io.rb @@ -238,6 +238,11 @@ assert('IO.sysopen, IO#sysread') do assert_raise(IOError) { io.sysread(1) } assert_raise(ArgumentError) { io.sysread(-1) } io.closed? + + fd = IO.sysopen $mrbtest_io_wfname, "w" + io = IO.new fd, "w" + assert_raise(IOError) { io.sysread(1) } + io.close end assert('IO.sysopen, IO#syswrite') do |
