[compiler-rt] r242350 - [probile] Fix memory leak introduced in r241824.

Alexey Samsonov vonosmas at gmail.com
Wed Jul 15 16:13:17 PDT 2015


I'm pretty sure we should, but let Diego verify that.

On Wed, Jul 15, 2015 at 4:09 PM, Hans Wennborg <hans at chromium.org> wrote:

> On Wed, Jul 15, 2015 at 3:50 PM, Alexey Samsonov <vonosmas at gmail.com>
> wrote:
> > Author: samsonov
> > Date: Wed Jul 15 17:50:39 2015
> > New Revision: 242350
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=242350&view=rev
> > Log:
> > [probile] Fix memory leak introduced in r241824.
> >
> > Modified:
> >     compiler-rt/trunk/lib/profile/InstrProfilingFile.c
> >
> > Modified: compiler-rt/trunk/lib/profile/InstrProfilingFile.c
> > URL:
> http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/profile/InstrProfilingFile.c?rev=242350&r1=242349&r2=242350&view=diff
> >
> ==============================================================================
> > --- compiler-rt/trunk/lib/profile/InstrProfilingFile.c (original)
> > +++ compiler-rt/trunk/lib/profile/InstrProfilingFile.c Wed Jul 15
> 17:50:39 2015
> > @@ -90,6 +90,7 @@ static void truncateCurrentFile(void) {
> >      char *Copy = malloc(strlen(Filename) + 1);
> >      strcpy(Copy, Filename);
> >      __llvm_profile_recursive_mkdir(Copy);
> > +    free(Copy);
>
> Should we merge this to the 3.7 branch?
>



-- 
Alexey Samsonov
vonosmas at gmail.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150715/a5d17f00/attachment.html>


More information about the llvm-commits mailing list