diff options
| author | Stéphane Wirtel <stephane@wirtel.be> | 2018-10-17 23:05:04 (GMT) |
|---|---|---|
| committer | Victor Stinner <vstinner@redhat.com> | 2018-10-17 23:05:04 (GMT) |
| commit | 74a8b6ea7e0a8508b13a1c75ec9b91febd8b5557 (patch) | |
| tree | 2ae2721beb480ede309e03b90202038b63c5a22c /Python/fileutils.c | |
| parent | 0f11a88622ceda93a6b7eed7db52a5eb8083445f (diff) | |
| download | cpython-74a8b6ea7e0a8508b13a1c75ec9b91febd8b5557.zip cpython-74a8b6ea7e0a8508b13a1c75ec9b91febd8b5557.tar.gz cpython-74a8b6ea7e0a8508b13a1c75ec9b91febd8b5557.tar.bz2 | |
bpo-24658: Fix read/write greater than 2 GiB on macOS (GH-1705)
On macOS, fix reading from and writing into a file with a size larger than 2 GiB.
Diffstat (limited to 'Python/fileutils.c')
| -rw-r--r-- | Python/fileutils.c | 24 |
1 files changed, 5 insertions, 19 deletions
diff --git a/Python/fileutils.c b/Python/fileutils.c index 0486f86..2c4061e 100644 --- a/Python/fileutils.c +++ b/Python/fileutils.c @@ -1471,18 +1471,9 @@ _Py_read(int fd, void *buf, size_t count) * handler raised an exception. */ assert(!PyErr_Occurred()); -#ifdef MS_WINDOWS - if (count > INT_MAX) { - /* On Windows, the count parameter of read() is an int */ - count = INT_MAX; - } -#else - if (count > PY_SSIZE_T_MAX) { - /* if count is greater than PY_SSIZE_T_MAX, - * read() result is undefined */ - count = PY_SSIZE_T_MAX; + if (count > _PY_READ_MAX) { + count = _PY_READ_MAX; } -#endif _Py_BEGIN_SUPPRESS_IPH do { @@ -1533,15 +1524,10 @@ _Py_write_impl(int fd, const void *buf, size_t count, int gil_held) depending on heap usage). */ count = 32767; } - else if (count > INT_MAX) - count = INT_MAX; -#else - if (count > PY_SSIZE_T_MAX) { - /* write() should truncate count to PY_SSIZE_T_MAX, but it's safer - * to do it ourself to have a portable behaviour. */ - count = PY_SSIZE_T_MAX; - } #endif + if (count > _PY_WRITE_MAX) { + count = _PY_WRITE_MAX; + } if (gil_held) { do { |