Skip to content

[3.7] bpo-33871: Fix os.sendfile(), os.writev(), os.readv(), etc. (GH-7931)#8583

Merged
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-9d57273-3.7
Jul 31, 2018
Merged

[3.7] bpo-33871: Fix os.sendfile(), os.writev(), os.readv(), etc. (GH-7931)#8583
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-9d57273-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented Jul 31, 2018

Copy link
Copy Markdown
Contributor
  • Fix integer overflow in os.readv(), os.writev(), os.preadv()
    and os.pwritev() and in os.sendfile() with headers or trailers
    arguments (on BSD-based OSes and MacOS).

  • Fix sending the part of the file in os.sendfile() on MacOS.
    Using the trailers argument could cause sending more bytes from
    the input file than was specified.

Thanks Ned Deily for testing on 32-bit MacOS.
(cherry picked from commit 9d57273)

Co-authored-by: Serhiy Storchaka storchaka@gmail.com

https://bugs.python.org/issue33871

…-7931)

* Fix integer overflow in os.readv(), os.writev(), os.preadv()
  and os.pwritev() and in os.sendfile() with headers or trailers
  arguments (on BSD-based OSes and MacOS).

* Fix sending the part of the file in os.sendfile() on MacOS.
  Using the trailers argument could cause sending more bytes from
  the input file than was specified.

Thanks Ned Deily for testing on 32-bit MacOS.
(cherry picked from commit 9d57273)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@serhiy-storchaka: Backport status check is done, and it's a success ✅ .

@miss-islington
miss-islington merged commit 3e4b688 into python:3.7 Jul 31, 2018
@miss-islington
miss-islington deleted the backport-9d57273-3.7 branch July 31, 2018 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bug An unexpected behavior, bug, or error type-security A security issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants