[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