From e36f44caa8d63e5643924e2f7f242174b4484fa9 Mon Sep 17 00:00:00 2001
From: nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>
Date: Fri, 1 Aug 2003 07:23:00 +0000
Subject: * io.c (rb_io_check_readable, rb_io_check_writable): ensure not  
 closed at first.

* io.c (rb_io_getline): check readable always.  (ruby-bugs:PR#1069)

* io.c (rb_io_each_byte): ditto.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@4262 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
---
 ChangeLog |  9 +++++++++
 io.c      | 11 +++++------
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index b35dd10a67..53e4c02a8a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+Fri Aug  1 16:22:57 2003  Nobuyoshi Nakada  <nobu@ruby-lang.org>
+
+	* io.c (rb_io_check_readable, rb_io_check_writable): ensure not
+	  closed at first.
+
+	* io.c (rb_io_getline): check readable always.  (ruby-bugs:PR#1069)
+
+	* io.c (rb_io_each_byte): ditto.
+
 Fri Aug  1 16:02:46 2003  Nobuyoshi Nakada  <nobu@ruby-lang.org>
 
 	* io.c (READ_DATA_PENDING_PTR): cast to get rid of warnings.
diff --git a/io.c b/io.c
index 536b8b2933..1256663f13 100644
--- a/io.c
+++ b/io.c
@@ -219,6 +219,7 @@ void
 rb_io_check_readable(fptr)
     OpenFile *fptr;
 {
+    rb_io_check_closed(fptr);
     if (!(fptr->mode & FMODE_READABLE)) {
 	rb_raise(rb_eIOError, "not opened for reading");
     }
@@ -236,6 +237,7 @@ void
 rb_io_check_writable(fptr)
     OpenFile *fptr;
 {
+    rb_io_check_closed(fptr);
     if (!(fptr->mode & FMODE_WRITABLE)) {
 	rb_raise(rb_eIOError, "not opened for writing");
     }
@@ -1007,6 +1009,7 @@ rb_io_getline(rs, fptr)
 {
     VALUE str = Qnil;
 
+    rb_io_check_readable(fptr);
     if (NIL_P(rs)) {
 	str = read_all(fptr, 0, Qnil);
     }
@@ -1082,7 +1085,6 @@ rb_io_gets_m(argc, argv, io)
 	rb_scan_args(argc, argv, "1", &rs);
     }
     GetOpenFile(io, fptr);
-    rb_io_check_readable(fptr);
     str = rb_io_getline(rs, fptr);
 
     if (!NIL_P(str)) {
@@ -1170,7 +1172,6 @@ rb_io_readlines(argc, argv, io)
 	rb_scan_args(argc, argv, "1", &rs);
     }
     GetOpenFile(io, fptr);
-    rb_io_check_readable(fptr);
     ary = rb_ary_new();
     while (!NIL_P(line = rb_io_getline(rs, fptr))) {
 	rb_ary_push(ary, line);
@@ -1195,7 +1196,6 @@ rb_io_each_line(argc, argv, io)
 	rb_scan_args(argc, argv, "1", &rs);
     }
     GetOpenFile(io, fptr);
-    rb_io_check_readable(fptr);
     while (!NIL_P(str = rb_io_getline(rs, fptr))) {
 	rb_yield(str);
     }
@@ -1211,10 +1211,10 @@ rb_io_each_byte(io)
     int c;
 
     GetOpenFile(io, fptr);
-    rb_io_check_readable(fptr);
-    f = fptr->f;
 
     for (;;) {
+	rb_io_check_readable(fptr);
+	f = fptr->f;
 	READ_CHECK(f);
 	TRAP_BEG;
 	c = getc(f);
@@ -3092,7 +3092,6 @@ argf_getline(argc, argv)
 	    rb_scan_args(argc, argv, "1", &rs);
 	}
 	GetOpenFile(current_file, fptr);
-	rb_io_check_readable(fptr);
 	line = rb_io_getline(rs, fptr);
     }
     if (NIL_P(line) && next_p != -1) {
-- 
cgit v1.2.3