[llvm] r212537 - Fix some Twine locals.

Reid Kleckner rnk at google.com
Wed Jul 9 16:15:36 PDT 2014


On Tue, Jul 8, 2014 at 7:55 AM, Benjamin Kramer <benny.kra at googlemail.com>
wrote:

> Author: d0k
> Date: Tue Jul  8 09:55:06 2014
> New Revision: 212537
>
> URL: http://llvm.org/viewvc/llvm-project?rev=212537&view=rev
> Log:
> Fix some Twine locals.
>
> Two of those are use after frees. Found by clang-tidy, fixed by me.
>

More like, "use after destroy" bugs, right?  I thought Twine doesn't do any
heap allocation, just stack allocation.

It's easy for these to bugs slip in when you have a compiler that doesn't
aggressively reuse stack space of dead temporaries.  *cough clang -O0*
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140709/9999b1d7/attachment.html>


More information about the llvm-commits mailing list