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

ICU-23011 Add else branch for remove possible CWE-476 #3338

Merged

markusicu merged 1 commit into unicode-org:main from wooffie:fix-ICU-23011

Jan 30, 2025

+4 −3

Contributor

wooffie commented Jan 21, 2025 •

edited by markusicu

Loading

Based on ICU-23011 I offer to add else branch to avoid nullptr dereference

Checklist

Required: Issue filed: ICU-23011
Required: The PR title must be prefixed with a JIRA Issue number. Example: "ICU-1234 Fix xyz"
Required: Each commit message must be prefixed with a JIRA Issue number. Example: "ICU-1234 Fix xyz"
Issue accepted (done by Technical Committee after discussion)
Tests included, if applicable
API docs and/or User Guide docs changed or added, if applicable


          ICU-23011 Add else branch for remove possible CWE-476

daf0422

markusicu self-assigned this

markusicu requested review from catamorphism and richgillam

January 30, 2025 17:43

markusicu approved these changes

View reviewed changes

markusicu merged commit d61ec78 into unicode-org:main

94 checks passed

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

Labels

None yet