Skip to content
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

[3.8] bpo-38400 Don't check for NULL linked list pointers in _PyObject_IsFreed (GH-16630) #18072

Merged
merged 1 commit into from Jan 19, 2020

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jan 19, 2020

Some objects like Py_None are not initialized with conventional means
that prepare the circular linked list pointers, leaving them unlinked
from the rest of the objects. For those objects, NULL pointers does
not mean that they are freed, so we need to skip the check in those
cases.
(cherry picked from commit 36e33c3)

Co-authored-by: Pablo Galindo Pablogsal@gmail.com

https://bugs.python.org/issue38400

…eed (pythonGH-16630)

Some objects like Py_None are not initialized with conventional means
that prepare the circular linked list pointers, leaving them unlinked
from the rest of the objects. For those objects, NULL pointers does
not mean that they are freed, so we need to skip the check in those
cases.
(cherry picked from commit 36e33c3)

Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>
@miss-islington
Copy link
Contributor Author

miss-islington commented Jan 19, 2020

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

@miss-islington miss-islington merged commit 4cdb758 into python:3.8 Jan 19, 2020
@miss-islington miss-islington deleted the backport-36e33c3-3.8 branch Jan 19, 2020
@miss-islington
Copy link
Contributor Author

miss-islington commented Jan 19, 2020

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

1 similar comment
@miss-islington
Copy link
Contributor Author

miss-islington commented Jan 19, 2020

@pablogsal: 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants