This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: coverage.c: Fix leak of da_file_name


On Wed, 2013-10-09 at 19:56 -0700, Ian Lance Taylor wrote:
> On Wed, Oct 9, 2013 at 5:18 PM, David Malcolm <dmalcolm@redhat.com> wrote:
> > coverage_init allocates da_file_name using XNEWVEC, but it is never
> > freed.  This shows up as a slow memory leak in my JIT code when
> > repeatedly invoking the compiler code within a single process.
> >
> > This patch frees it in coverage_finish with XDELETEVEC (and NULLs it to
> > be sure).
> >
> > Bootstrapped and regtested against trunk on x86_64-unknown-linux.
> >
> > OK for trunk?
> 
> This is OK.

Thanks; committed to trunk as r203388.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]