Skip to content
Snippets Groups Projects

Encoding: Reformulate to avoid unsigned int overflow

Merged Sean McBride requested to merge seanm/kwsys:ubsan-well-defined into master
All threads resolved!

Playing with clang UBSan unsigned-integer-overflow I found just 2 warnings in KWSys.

They are not undefined behavior, but often unintentional.

A reformulation to avoid the overflow was pretty straightforward.

Also noticed and changed &v[0] idiom to v.data().

Edited by Brad King

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading