diff options
author | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2017-10-27 18:37:23 +0000 |
---|---|---|
committer | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2017-10-27 18:37:23 +0000 |
commit | 524e6608773198c1359fda8b1c570aaf85976572 (patch) | |
tree | f6017349dbe50c8991e17f5bb6a05666fe27f73f /io.c | |
parent | c2be8192503215860ec297d62cefe45f1a359bb3 (diff) |
io.c: fix IO.copy_stream on O_APPEND destination on Linux
Linux copy_file_range(2) fails with EBADF if the destination FD
has O_APPEND set. Preserve existing (Ruby <= 2.4) behavior by
falling back to alternative copy mechanisms if this is the case
(instead of raising Errno::EBADF).
* io.c (nogvl_copy_file_range): do not raise on O_APPEND dst
* test/ruby/test_io.rb (test_copy_stream_append): new test
[Feature #13867]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60490 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'io.c')
-rw-r--r-- | io.c | 10 |
1 files changed, 10 insertions, 0 deletions
@@ -10789,6 +10789,16 @@ nogvl_copy_file_range(struct copy_stream_struct *stp) if (nogvl_copy_stream_wait_write(stp) == -1) return -1; goto retry_copy_file_range; + case EBADF: + { + int e = errno; + int flags = fcntl(stp->dst_fd, F_GETFL); + + if (flags != -1 && flags & O_APPEND) { + return 0; + } + errno = e; + } } stp->syserr = "copy_file_range"; stp->error_no = errno; |