diff --git a/ChangeLog b/ChangeLog index aa966b7da8..0962f6af63 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,4 +1,9 @@ -Wed May 22 15:17:30 2013 Nobuyoshi Nakada +Wed May 22 15:18:59 2013 Nobuyoshi Nakada + + * win32/win32.c (setup_overlapped): check the error code in addition + to the result of SetFilePointer() to determine if an error occurred, + because INVALID_SET_FILE_POINTER is a valid value. + [ruby-core:55098] [Bug #8431] * win32/win32.c (setup_overlapped, finish_overlapped): extract from rb_w32_read() and rb_w32_write(). diff --git a/test/ruby/test_io.rb b/test/ruby/test_io.rb index d44dd31e71..ca741f8a23 100644 --- a/test/ruby/test_io.rb +++ b/test/ruby/test_io.rb @@ -2673,4 +2673,15 @@ End IO.select(tempfiles) }, bug8080 end + + def test_seek_32bit_boundary + bug8431 = '[ruby-core:55098] [Bug #8431]' + make_tempfile {|t| + assert_ruby_status(["-e", <<-"end;", t.path], "", bug8431) + f = ARGF.to_io + f.seek(0xffff_ffff) + f.read(1) + end; + } + end end diff --git a/win32/win32.c b/win32/win32.c index 74d13960bc..b7eb4c62f2 100644 --- a/win32/win32.c +++ b/win32/win32.c @@ -6028,8 +6028,11 @@ setup_overlapped(OVERLAPPED *ol, int fd) #define INVALID_SET_FILE_POINTER ((DWORD)-1) #endif if (low == INVALID_SET_FILE_POINTER) { - errno = map_errno(GetLastError()); - return -1; + DWORD err = GetLastError(); + if (err != NO_ERROR) { + errno = map_errno(err); + return -1; + } } ol->Offset = low; ol->OffsetHigh = high;