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

Change UBSAN for undefined instead of address #2964

Merged
merged 1 commit into from
May 17, 2023
Merged

Change UBSAN for undefined instead of address #2964

merged 1 commit into from
May 17, 2023

Conversation

byrnHDF
Copy link
Contributor

@byrnHDF byrnHDF commented May 16, 2023

No description provided.

@byrnHDF byrnHDF added Priority - 1. High 🔼 These are important issues that should be resolved in the next release Component - C Library Core C library issues (usually in the src directory) Type - Bug / Bugfix Please report security issues to [email protected] instead of creating an issue on GitHub Type - Improvement Improvements that don't add a new feature or functionality labels May 16, 2023
@byrnHDF byrnHDF self-assigned this May 16, 2023
@byrnHDF
Copy link
Contributor Author

byrnHDF commented May 16, 2023

Addresses #2947

@jhendersonHDF jhendersonHDF linked an issue May 16, 2023 that may be closed by this pull request
HDF_NO_UBSAN should (also) disable UB sanitizer #2947
Closed
@lrknox lrknox changed the title Change UBSAN for undefined instaed of address Change UBSAN for undefined instead of address May 17, 2023
@lrknox lrknox merged commit c6edc88 into HDFGroup:develop May 17, 2023
brtnfld pushed a commit to brtnfld/hdf5 that referenced this pull request May 17, 2023
@byrnHDF @brtnfld
Change UBSAN for undefined instaed of address (HDFGroup#2964)
c421f0f
@byrnHDF byrnHDF deleted the develop-ubsan branch May 23, 2023 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jhendersonHDF jhendersonHDF jhendersonHDF approved these changes

@qkoziol qkoziol qkoziol approved these changes

@lrknox lrknox Awaiting requested review from lrknox lrknox is a code owner

@derobins derobins Awaiting requested review from derobins derobins is a code owner

@fortnern fortnern Awaiting requested review from fortnern fortnern is a code owner

@vchoi-hdfgroup vchoi-hdfgroup Awaiting requested review from vchoi-hdfgroup vchoi-hdfgroup is a code owner

@bmribler bmribler Awaiting requested review from bmribler bmribler is a code owner

@glennsong09 glennsong09 Awaiting requested review from glennsong09 glennsong09 is a code owner

@mattjala mattjala Awaiting requested review from mattjala mattjala is a code owner

@brtnfld brtnfld Awaiting requested review from brtnfld brtnfld is a code owner

Assignees

@byrnHDF byrnHDF

Labels
Component - C Library Core C library issues (usually in the src directory) Priority - 1. High 🔼 These are important issues that should be resolved in the next release Type - Bug / Bugfix Please report security issues to [email protected] instead of creating an issue on GitHub Type - Improvement Improvements that don't add a new feature or functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

HDF_NO_UBSAN should (also) disable UB sanitizer
4 participants
@byrnHDF @qkoziol @jhendersonHDF @lrknox