Skip to content

gh-111178: Set function=False by default in support.check_sanitizer() - #161640

Open
vstinner wants to merge 1 commit into
python:mainfrom
vstinner:sanitize_function
Open

gh-111178: Set function=False by default in support.check_sanitizer()#161640
vstinner wants to merge 1 commit into
python:mainfrom
vstinner:sanitize_function

Conversation

@vstinner

@vstinner vstinner commented Nov 8, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

Update also libregrtest.utils to detect function sanitizer (-fsanitize=function).

@vstinner
pythongh-111178: Set function=False by default in support.check_sanit…
2519b43 
…izer()

Update also libregrtest.utils to detect function sanitizer
(-fsanitize=function).
@vstinner vstinner added skip news needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes labels Nov 8, 2026
@bedevere-app bedevere-app Bot mentioned this pull request Nov 8, 2026
Closed
@vstinner

vstinner commented Nov 8, 2026

Copy link
Copy Markdown
Member Author

The function parameter was added to check_sanitizer() by commit 643dd51. I made the change, but I don't remember why it's function=True by default. IMO it's a mistake and it should be function=False by default, as other parameters.

See also #111178.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

awaiting core review needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vstinner