diff --git a/ChangeLog b/ChangeLog index fbaada6e4d..11a55162f7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +Tue Aug 30 14:53:34 2016 NAKAMURA Usaku + + * io.c (nogvl_fsync, nogvl_fdatasync): on Windows, just ignore if the + fd is associated to non-disk device. if call fsync and/or fdatasync + with such fds, it causes Errno::EBADF exception and the behavior is + incomatible with ruby 2.1 and earlier unintendedly introduced. + Tue Aug 30 03:38:35 2016 NARUSE, Yui * vm_dump.c (backtrace): use rip in the saved context for the case diff --git a/io.c b/io.c index fbd24f6046..137bc77fb1 100644 --- a/io.c +++ b/io.c @@ -1533,6 +1533,10 @@ nogvl_fsync(void *ptr) { rb_io_t *fptr = ptr; +#ifdef _WIN32 + if (GetFileType((HANDLE)rb_w32_get_osfhandle(fptr->fd)) != FILE_TYPE_DISK) + return 0; +#endif return (VALUE)fsync(fptr->fd); } #endif @@ -1935,6 +1939,10 @@ nogvl_fdatasync(void *ptr) { rb_io_t *fptr = ptr; +#ifdef _WIN32 + if (GetFileType((HANDLE)rb_w32_get_osfhandle(fptr->fd)) != FILE_TYPE_DISK) + return 0; +#endif return (VALUE)fdatasync(fptr->fd); }