Skip to content

Conversation

@gpshead
Copy link
Member

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

@gpsheadgpshead merged commit e5796c4 into python:masterDec 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 <greg@krypto.org>
@bedevere-bot
Copy link

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

@gpsheadgpshead 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 <greg@krypto.org>
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 freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testsTests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@gpshead@miss-islington@bedevere-bot@the-knights-who-say-ni