Skip to content

Conversation

@gpshead
Copy link
Member

@gpshead gpshead commented Dec 31, 2018

The clang memory sanitizers default behavior is to crash when a malloc that cannot succeed is called rather than returning an error. Lets just skip these tests under msan; not much value there.

https://bugs.python.org/issue35214

@gpshead gpshead merged commit e5796c4 into python:master Dec 31, 2018
@miss-islington
Copy link
Contributor

Thanks @gpshead for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@bedevere-bot
Copy link

@gpshead: Please replace # with GH- in the commit message next time. Thanks!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 31, 2018
…ythonGH-11385)

* skip test_constructor under msan.

* fix the others as well.

* reuse existing related news entry.

* typo fix
(cherry picked from commit e5796c4)

Co-authored-by: Gregory P. Smith <[email protected]>
@bedevere-bot
Copy link

GH-11388 is a backport of this pull request to the 3.7 branch.

@gpshead gpshead deleted the msan_test_io branch December 31, 2018 04:18
miss-islington added a commit that referenced this pull request Dec 31, 2018
…H-11385)

* skip test_constructor under msan.

* fix the others as well.

* reuse existing related news entry.

* typo fix
(cherry picked from commit e5796c4)

Co-authored-by: Gregory P. Smith <[email protected]>
csabella pushed a commit to csabella/cpython that referenced this pull request Dec 31, 2018
…ython#11385)

* skip test_constructor under msan.

* fix the others as well.

* reuse existing related news entry.

* typo fix
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