fix windows pread and pwrite bug

reading without a byte offset specified can affect absolute reads afterward, this is a workaround (thanks windows...)

1 files changed, 7 insertions, 7 deletions

diff --git a/source/core/StarFile_windows.cpp b/source/core/StarFile_windows.cpp
index 0b5c286..6352bb7 100644
--- a/source/core/StarFile_windows.cpp
+++ b/source/core/StarFile_windows.cpp
@@ -18,13 +18,11 @@
 
 namespace Star {
 
-namespace {
-  OVERLAPPED makeOverlapped(StreamOffset offset) {
-    OVERLAPPED overlapped = {};
-    overlapped.Offset = offset;
-    overlapped.OffsetHigh = offset >> 32;
-    return overlapped;
-  }
+OVERLAPPED makeOverlapped(StreamOffset offset) {
+  OVERLAPPED overlapped = {};
+  overlapped.Offset = offset;
+  overlapped.OffsetHigh = offset >> 32;
+  return overlapped;
 }
 
 String File::convertDirSeparators(String const& path) {
@@ -378,6 +376,7 @@ size_t File::pread(void* f, char* data, size_t len, StreamOffset position) {
   DWORD numRead = 0;
   OVERLAPPED overlapped = makeOverlapped(position);
   int ret = ReadFile(file, data, len, &numRead, &overlapped);
+  fseek(f, -(StreamOffset)numRead, IOSeek::Relative);
   if (ret == 0) {
     auto err = GetLastError();
     if (err != ERROR_IO_PENDING)
@@ -392,6 +391,7 @@ size_t File::pwrite(void* f, char const* data, size_t len, StreamOffset position
   DWORD numWritten = 0;
   OVERLAPPED overlapped = makeOverlapped(position);
   int ret = WriteFile(file, data, len, &numWritten, &overlapped);
+  fseek(f, -(StreamOffset)numWritten, IOSeek::Relative);
   if (ret == 0) {
     auto err = GetLastError();
     if (err != ERROR_IO_PENDING)