Skip to content

[3.7] bpo-31731: Fix test_io.check_interrupted_write() (GH-11225) - #11226

Merged
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-05c9d31-3.7
Dec 18, 2018
Merged

miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-05c9d31-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented Dec 18, 2018 •

Copy link
Copy Markdown
Contributor

Fix a race condition in check_interrupted_write() of test_io:
create directly the thread with SIGALRM signal blocked,
rather than blocking the signal later from the thread. Previously, it
was possible that the thread gets the signal before the signal is
blocked.
(cherry picked from commit 05c9d31)

Co-authored-by: Victor Stinner [email protected]

https://bugs.python.org/issue31731

Fix a race condition in check_interrupted_write() of test_io:
create directly the thread with SIGALRM signal blocked,
rather than blocking the signal later from the thread. Previously, it
was possible that the thread gets the signal before the signal is
blocked.
(cherry picked from commit 05c9d31)

Co-authored-by: Victor Stinner <[email protected]>

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good bot.

@miss-islington

Copy link
Copy Markdown
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

@miss-islington
miss-islington merged commit 729fc5d into python:3.7 Dec 18, 2018
@miss-islington
miss-islington deleted the backport-05c9d31-3.7 branch December 18, 2018 23:10
@miss-islington

Copy link
Copy Markdown
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants