[proxy] github.com — ← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public

Conversation

Copy link
Contributor

miss-islington commented Nov 30, 2018 •
edited by bedevere-bot
Loading

Fix an undefined behaviour in the pthread implementation of
PyThread_start_new_thread(): add a function wrapper to always return
NULL.

Add pythread_callback struct and pythread_wrapper() to thread_pthread.h.
(cherry picked from commit 9eea6ea)

Co-authored-by: Siddhesh Poyarekar siddhesh.poyarekar@gmail.com

https://bugs.python.org/issue33015

bpo-33015: Fix UB in pthread PyThread_start_new_thread (pythonGH-6008)
fb6feaa 
Fix an undefined behaviour in the pthread implementation of
PyThread_start_new_thread(): add a function wrapper to always return
NULL.

Add pythread_callback struct and pythread_wrapper() to thread_pthread.h.
(cherry picked from commit 9eea6ea)

Co-authored-by: Siddhesh Poyarekar <siddhesh.poyarekar@gmail.com>
bedevere-bot mentioned this pull request Nov 30, 2018
Merged
vstinner approved these changes Nov 30, 2018
View reviewed changes
Copy link
Member

vstinner left a comment

Choose a reason for hiding this comment

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

LGTM, good bot.

Copy link
Contributor Author

miss-islington commented Nov 30, 2018

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

miss-islington merged commit b135535 into python:3.7 Nov 30, 2018
miss-islington deleted the backport-9eea6ea-3.7 branch November 30, 2018 15:32
Copy link
Contributor Author

miss-islington commented Nov 30, 2018

@siddhesh and @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

Reviewers

vstinner vstinner approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants