◐ Shell
reader mode source ↗
Skip to content

[3.7] bpo-33015: Fix UB in pthread PyThread_start_new_thread (GH-6008)#10821

Merged
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-9eea6ea-3.7
Nov 30, 2018
Merged

[3.7] bpo-33015: Fix UB in pthread PyThread_start_new_thread (GH-6008)#10821
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-9eea6ea-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented Nov 30, 2018

Copy link
Copy Markdown
Contributor

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

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>

@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.

Hide comment

LGTM, good bot.

@miss-islington

Copy link
Copy Markdown
Contributor Author

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

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

Copy link
Copy Markdown
Contributor Author

@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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants