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.

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

ICU-22224 Fix undefine behavior in rbbi constructor #4

Merged
merged 1 commit into from
Feb 27, 2023

Conversation

aheninger
Copy link

@aheninger aheninger commented Feb 26, 2023

Fix the undefined behavior in the default RuleBasedBreakIterator constructor that was caused by referencing a member of a not-yet-constructed object.

Checklist
  • Required: Issue filed: https://unicode-org.atlassian.net/browse/ICU-22224
  • Required: The PR title must be prefixed with a JIRA Issue number.
  • Required: The PR description must include the link to the Jira Issue, for example by completing the URL in the first checklist item
  • Required: Each commit message must be prefixed with a JIRA Issue number.
  • Issue accepted (done by Technical Committee after discussion)
  • Tests included, if applicable
  • API docs and/or User Guide docs changed or added, if applicable

Fix the undefined behavior in the default RuleBasedBreakIterator constructor
that was caused by referencing a member of a not-yet-constructed object.
@FrankYFTang FrankYFTang merged commit c9b65d7 into FrankYFTang:ICU-22224-ubsan Feb 27, 2023
@aheninger aheninger deleted the ubsan-rbbi branch February 28, 2023 01:06
FrankYFTang added a commit that referenced this pull request Sep 25, 2024
# This is the 1st commit message:

ICU-22767 Fix GCC warning and turn warning to errors

# This is the commit message #2:

ICU-22716 use pre-existing task

# This is the commit message #3:

ICU-22716 Fix

# This is the commit message #4:

ICU-22716 Fix unitialization

# This is the commit message unicode-org#5:

Update icu4c/source/common/ushape.cpp

Co-authored-by: Fredrik Roubert <fredrik@roubert.name>
# This is the commit message unicode-org#6:

ICU-22716 change macro

# This is the commit message unicode-org#7:

ICU-22716 Add document about the macro

# This is the commit message unicode-org#8:

ICU-22716 Addres review feedback
FrankYFTang added a commit that referenced this pull request Sep 25, 2024
# This is the 1st commit message:

ICU-22767 Fix GCC warning and turn warning to errors

# This is the commit message #2:

ICU-22716 use pre-existing task

# This is the commit message #3:

ICU-22716 Fix

# This is the commit message #4:

ICU-22716 Fix unitialization

# This is the commit message unicode-org#5:

Update icu4c/source/common/ushape.cpp

Co-authored-by: Fredrik Roubert <fredrik@roubert.name>
# This is the commit message unicode-org#6:

ICU-22716 change macro

# This is the commit message unicode-org#7:

ICU-22716 Add document about the macro

# This is the commit message unicode-org#8:

ICU-22716 Addres review feedback

# This is the commit message unicode-org#9:

ICU-22767 Fix GCC warning and turn warning to errors

# This is the commit message unicode-org#10:

ICU-22716 use pre-existing task

# This is the commit message unicode-org#11:

ICU-22716 Fix

# This is the commit message unicode-org#12:

ICU-22716 Fix unitialization

# This is the commit message unicode-org#13:

Update icu4c/source/common/ushape.cpp

Co-authored-by: Fredrik Roubert <fredrik@roubert.name>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants