mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
* io.c (rb_io_close_read): replaces fptr with the tied writer if
duplex. * io.c (rb_io_close_write): unties the tied IO for writing if duplex. [ruby-dev:33532] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@15351 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
d7f5b291be
commit
6f4d6b1650
2 changed files with 28 additions and 5 deletions
|
@ -1,3 +1,11 @@
|
||||||
|
Thu Jan 31 06:24:22 2008 Nobuyoshi Nakada <nobu@ruby-lang.org>
|
||||||
|
|
||||||
|
* io.c (rb_io_close_read): replaces fptr with the tied writer if
|
||||||
|
duplex.
|
||||||
|
|
||||||
|
* io.c (rb_io_close_write): unties the tied IO for writing if duplex.
|
||||||
|
[ruby-dev:33532]
|
||||||
|
|
||||||
Thu Jan 31 02:22:04 2008 Yukihiro Matsumoto <matz@ruby-lang.org>
|
Thu Jan 31 02:22:04 2008 Yukihiro Matsumoto <matz@ruby-lang.org>
|
||||||
|
|
||||||
* io.c (open_key_args): allow encoding key to take two encoding
|
* io.c (open_key_args): allow encoding key to take two encoding
|
||||||
|
|
25
io.c
25
io.c
|
@ -2298,7 +2298,7 @@ rb_io_getc(VALUE io)
|
||||||
else if (MBCLEN_NEEDMORE_P(r)) {
|
else if (MBCLEN_NEEDMORE_P(r)) {
|
||||||
str = rb_str_new(fptr->rbuf+fptr->rbuf_off, fptr->rbuf_len);
|
str = rb_str_new(fptr->rbuf+fptr->rbuf_off, fptr->rbuf_len);
|
||||||
fptr->rbuf_len = 0;
|
fptr->rbuf_len = 0;
|
||||||
getc_needmore:
|
getc_needmore:
|
||||||
if (io_fillbuf(fptr) != -1) {
|
if (io_fillbuf(fptr) != -1) {
|
||||||
rb_str_cat(str, fptr->rbuf+fptr->rbuf_off, 1);
|
rb_str_cat(str, fptr->rbuf+fptr->rbuf_off, 1);
|
||||||
fptr->rbuf_off++;
|
fptr->rbuf_off++;
|
||||||
|
@ -2784,7 +2784,14 @@ rb_io_close_read(VALUE io)
|
||||||
|
|
||||||
write_io = GetWriteIO(io);
|
write_io = GetWriteIO(io);
|
||||||
if (io != write_io) {
|
if (io != write_io) {
|
||||||
|
rb_io_t *wfptr;
|
||||||
fptr_finalize(fptr, Qfalse);
|
fptr_finalize(fptr, Qfalse);
|
||||||
|
GetOpenFile(write_io, wfptr);
|
||||||
|
if (fptr->refcnt < LONG_MAX) {
|
||||||
|
wfptr->refcnt++;
|
||||||
|
RFILE(io)->fptr = wfptr;
|
||||||
|
rb_io_fptr_finalize(fptr);
|
||||||
|
}
|
||||||
return Qnil;
|
return Qnil;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -2817,12 +2824,13 @@ static VALUE
|
||||||
rb_io_close_write(VALUE io)
|
rb_io_close_write(VALUE io)
|
||||||
{
|
{
|
||||||
rb_io_t *fptr;
|
rb_io_t *fptr;
|
||||||
|
VALUE write_io;
|
||||||
|
|
||||||
if (rb_safe_level() >= 4 && !OBJ_TAINTED(io)) {
|
if (rb_safe_level() >= 4 && !OBJ_TAINTED(io)) {
|
||||||
rb_raise(rb_eSecurityError, "Insecure: can't close");
|
rb_raise(rb_eSecurityError, "Insecure: can't close");
|
||||||
}
|
}
|
||||||
io = GetWriteIO(io);
|
write_io = GetWriteIO(io);
|
||||||
GetOpenFile(io, fptr);
|
GetOpenFile(write_io, fptr);
|
||||||
if (is_socket(fptr->fd, fptr->path)) {
|
if (is_socket(fptr->fd, fptr->path)) {
|
||||||
#ifndef SHUT_WR
|
#ifndef SHUT_WR
|
||||||
# define SHUT_WR 1
|
# define SHUT_WR 1
|
||||||
|
@ -2831,14 +2839,21 @@ rb_io_close_write(VALUE io)
|
||||||
rb_sys_fail(fptr->path);
|
rb_sys_fail(fptr->path);
|
||||||
fptr->mode &= ~FMODE_WRITABLE;
|
fptr->mode &= ~FMODE_WRITABLE;
|
||||||
if (!(fptr->mode & FMODE_READABLE))
|
if (!(fptr->mode & FMODE_READABLE))
|
||||||
return rb_io_close(io);
|
return rb_io_close(write_io);
|
||||||
return Qnil;
|
return Qnil;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (fptr->mode & FMODE_READABLE) {
|
if (fptr->mode & FMODE_READABLE) {
|
||||||
rb_raise(rb_eIOError, "closing non-duplex IO for writing");
|
rb_raise(rb_eIOError, "closing non-duplex IO for writing");
|
||||||
}
|
}
|
||||||
return rb_io_close(io);
|
|
||||||
|
rb_io_close(write_io);
|
||||||
|
if (io != write_io) {
|
||||||
|
GetOpenFile(io, fptr);
|
||||||
|
fptr->tied_io_for_writing = 0;
|
||||||
|
fptr->mode &= ~FMODE_DUPLEX;
|
||||||
|
}
|
||||||
|
return Qnil;
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
|
Loading…
Add table
Reference in a new issue