Commit d9fc85db authored by Victor Stinner's avatar Victor Stinner

(merge 3.2) Issue #9611, #9015: FileIO.read() clamps the length to INT_MAX on Windows.

parents 9797e29f c655a726
...@@ -10,6 +10,8 @@ What's New in Python 3.3 Alpha 1? ...@@ -10,6 +10,8 @@ What's New in Python 3.3 Alpha 1?
Core and Builtins Core and Builtins
----------------- -----------------
- Issue #9611, #9015: FileIO.read() clamps the length to INT_MAX on Windows.
- Issue #9642: Uniformize the tests on the availability of the mbcs codec, add - Issue #9642: Uniformize the tests on the availability of the mbcs codec, add
a new HAVE_MBCS define. a new HAVE_MBCS define.
...@@ -1327,7 +1329,7 @@ Core and Builtins ...@@ -1327,7 +1329,7 @@ Core and Builtins
(length bigger than 2^31-1 bytes). (length bigger than 2^31-1 bytes).
- Issue #9015, #9611: FileIO.readinto(), FileIO.write(), os.write() and - Issue #9015, #9611: FileIO.readinto(), FileIO.write(), os.write() and
stdprinter.write() clamp the length to 2^31-1 on Windows. stdprinter.write() clamp the length to INT_MAX on Windows.
- Issue #8278: On Windows and with a NTFS filesystem, os.stat() and os.utime() - Issue #8278: On Windows and with a NTFS filesystem, os.stat() and os.utime()
can now handle dates after 2038. can now handle dates after 2038.
......
...@@ -687,6 +687,10 @@ fileio_read(fileio *self, PyObject *args) ...@@ -687,6 +687,10 @@ fileio_read(fileio *self, PyObject *args)
return fileio_readall(self); return fileio_readall(self);
} }
#if defined(MS_WIN64) || defined(MS_WINDOWS)
if (size > INT_MAX)
size = INT_MAX;
#endif
bytes = PyBytes_FromStringAndSize(NULL, size); bytes = PyBytes_FromStringAndSize(NULL, size);
if (bytes == NULL) if (bytes == NULL)
return NULL; return NULL;
...@@ -695,7 +699,11 @@ fileio_read(fileio *self, PyObject *args) ...@@ -695,7 +699,11 @@ fileio_read(fileio *self, PyObject *args)
if (_PyVerify_fd(self->fd)) { if (_PyVerify_fd(self->fd)) {
Py_BEGIN_ALLOW_THREADS Py_BEGIN_ALLOW_THREADS
errno = 0; errno = 0;
#if defined(MS_WIN64) || defined(MS_WINDOWS)
n = read(self->fd, ptr, (int)size);
#else
n = read(self->fd, ptr, size); n = read(self->fd, ptr, size);
#endif
Py_END_ALLOW_THREADS Py_END_ALLOW_THREADS
} else } else
n = -1; n = -1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment