This is an archive of the discontinued LLVM Phabricator instance.

[scudo] Fix String DCHECK
ClosedPublic

Authored by cryptoad on Jun 4 2021, 1:38 PM.

Details

Summary

This resolves an issue tripping a DCHECK, as I was checking for the
capacity and not the size. We don't need to 0-init the Vector as it's
done already, and make sure we only 0-out the string on clear if it's
not empty.

Diff Detail

Event Timeline

cryptoad requested review of this revision.Jun 4 2021, 1:38 PM
cryptoad created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJun 4 2021, 1:38 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
hctim accepted this revision.Jun 4 2021, 1:41 PM

LGTM

This revision is now accepted and ready to land.Jun 4 2021, 1:41 PM
This revision was landed with ongoing or failed builds.Jun 4 2021, 1:42 PM
This revision was automatically updated to reflect the committed changes.