This is an archive of the discontinued LLVM Phabricator instance.

Fix memory leak in MicrosoftDemangleNodes's Node::toString
ClosedPublic

Authored by teemperor on Apr 20 2021, 5:01 AM.

Details

Summary

The buffer we turn into a std::string here is malloc'd and should be
free'd before we return from this function.

Follow up to LLDB leak fixes such as D100806.

Diff Detail

Event Timeline

teemperor created this revision.Apr 20 2021, 5:01 AM
teemperor requested review of this revision.Apr 20 2021, 5:01 AM
mstorsjo accepted this revision.Apr 20 2021, 5:07 AM

Oops, and thanks for fixing it!

This revision is now accepted and ready to land.Apr 20 2021, 5:07 AM
rupprecht accepted this revision.Apr 20 2021, 2:28 PM
MaskRay accepted this revision.Apr 20 2021, 3:09 PM