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.

Sign up for GitHub

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

Jump to bottom

Test with address sanitizer to catch memory safety issues #118

Merged
merged 1 commit into from
Nov 8, 2022
Merged

Test with address sanitizer to catch memory safety issues #118

merged 1 commit into from
Nov 8, 2022

Conversation

pcolberg
Copy link
Contributor

@pcolberg pcolberg commented May 6, 2022 •
edited
Loading

This enables address sanitizer for CI debug builds on all distributions,
which has proven useful in tracing undefined behaviour that only occurs
on an arbitrary but reproducible subset of distributions.

See #194

@pcolberg pcolberg added the enhancement New feature or request label May 6, 2022
@pcolberg pcolberg self-assigned this May 6, 2022
@pcolberg pcolberg marked this pull request as draft May 6, 2022 23:12
@pcolberg pcolberg linked an issue May 6, 2022 that may be closed by this pull request
Build with address sanitizer #117
Closed
2 tasks
@pcolberg pcolberg mentioned this pull request May 7, 2022
Merged
@pcolberg pcolberg force-pushed the asan branch 2 times, most recently from 61269ff to e473677 Compare May 7, 2022 01:04
@pcolberg pcolberg force-pushed the asan branch 2 times, most recently from 75b049c to c438265 Compare August 18, 2022 23:16
@pcolberg pcolberg force-pushed the asan branch 5 times, most recently from d08b931 to 58ce094 Compare October 28, 2022 22:37
@pcolberg pcolberg mentioned this pull request Oct 28, 2022
Merged
@pcolberg pcolberg mentioned this pull request Oct 28, 2022
Merged
@pcolberg pcolberg added this to the 2023.1 milestone Oct 28, 2022
@pcolberg pcolberg force-pushed the asan branch 11 times, most recently from 30962fb to 21fae75 Compare November 2, 2022 02:21
@pcolberg pcolberg mentioned this pull request Nov 2, 2022
Closed
@pcolberg pcolberg mentioned this pull request Nov 7, 2022
Merged
@pcolberg
workflows: test with address sanitizer to catch memory safety issues
d7b4a46 
This enables address sanitizer for CI debug builds on all distributions,
which has proven useful in tracing undefined behaviour that only occurs
on an arbitrary but reproducible subset of distributions.

See intel#194

Signed-off-by: Peter Colberg <peter.colberg@intel.com>
@pcolberg pcolberg force-pushed the asan branch 2 times, most recently from 1d7d74b to d7b4a46 Compare November 7, 2022 23:37
@pcolberg pcolberg marked this pull request as ready for review November 8, 2022 01:48
@pcolberg pcolberg requested a review from zibaiwan November 8, 2022 01:48
zibaiwan
zibaiwan approved these changes Nov 8, 2022
View reviewed changes
Copy link
Contributor

@zibaiwan zibaiwan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Peter!

@pcolberg pcolberg merged commit 91fcfd3 into intel:main Nov 8, 2022
@pcolberg pcolberg deleted the asan branch November 8, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zibaiwan zibaiwan zibaiwan approved these changes

Assignees

@pcolberg pcolberg

Labels
enhancement New feature or request
Projects
None yet
Milestone
2023.1
Development

Successfully merging this pull request may close these issues.

Build with address sanitizer
2 participants
@pcolberg @zibaiwan