New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
io.FileIO.readall() is not 64-bit safe on Windows #60571
Comments
The changeset 374dc910db33 fixed FileIO.readinto(), FileIO.write() and os.write() on Windows for issues bpo-9015, bpo-9611, but FileIO.readall() was not fixed and it has a similar bug. It uses "int n;" to store the result of read(), so FileIO.readall() may also fail to read a file longer than 2 GB on other platforms. Attached patch should fix both issues. |
Looks ok to me. Perhaps we could add a bigmem test for that (in test_fileio)? |
New changeset 9aba9ad6c15b by Victor Stinner in branch '3.2': New changeset 5f96e4619ceb by Victor Stinner in branch '3.3': New changeset d81d4b3059e4 by Victor Stinner in branch 'default': New changeset f26c91bf61bf by Victor Stinner in branch '2.7': |
Well, I'm too lazy to write such test. Instead of leaving the issue open for weeks, I prefer to commit my patch with no new test (sorry!). |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: