summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--Misc/NEWS2
-rw-r--r--Modules/_io/fileio.c13
2 files changed, 13 insertions, 2 deletions
diff --git a/Misc/NEWS b/Misc/NEWS
index 9e629e4..b00dc53 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -10,6 +10,8 @@ What's New in Python 3.4.0 Alpha 1?
Core and Builtins
-----------------
+- Issue #16367: Fix FileIO.readall() on Windows for files larger than 2 GB.
+
- Issue #16761: Calling int() with base argument only now raises TypeError.
- Issue #16759: Support the full DWORD (unsigned long) range in Reg2Py
diff --git a/Modules/_io/fileio.c b/Modules/_io/fileio.c
index beacd9f..34425b7 100644
--- a/Modules/_io/fileio.c
+++ b/Modules/_io/fileio.c
@@ -602,7 +602,7 @@ fileio_readall(fileio *self)
#endif
PyObject *result;
Py_ssize_t total = 0;
- int n;
+ Py_ssize_t n;
size_t newsize;
if (self->fd < 0)
@@ -651,9 +651,18 @@ fileio_readall(fileio *self)
}
Py_BEGIN_ALLOW_THREADS
errno = 0;
+ n = newsize - total;
+#if defined(MS_WIN64) || defined(MS_WINDOWS)
+ if (n > INT_MAX)
+ n = INT_MAX;
+ n = read(self->fd,
+ PyBytes_AS_STRING(result) + total,
+ (int)n);
+#else
n = read(self->fd,
PyBytes_AS_STRING(result) + total,
- newsize - total);
+ n);
+#endif
Py_END_ALLOW_THREADS
if (n == 0)
break;