Re: trivial one-off memory leak in guc-file.l ParseConfigFile

From: Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>
To: didier <did447(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: trivial one-off memory leak in guc-file.l ParseConfigFile
Date: 2013-11-15 18:20:10
Message-ID: 528665DA.2070302@vmware.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 22.09.2013 22:40, didier wrote:
> Hi
>
> fix a small memory leak in guc-file.l ParseConfigFile
>
> AbsoluteConfigLocation() return a strdup string but it's never free or
> referenced outside ParseConfigFile
>
> Courtesy Valgrind and Noah Misch MEMPOOL work.

I spotted and fixed this some time ago while fixing another leak, see
http://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=138184adc5f7c60c184972e4d23f8cdb32aed77d.
I didn't realize you had already reported it back then.

So, I've marked this as committed in the commitfest app. But thanks for
the report anyway.

- Heikki

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2013-11-15 18:21:02 Re: Sequence Access Method WIP
Previous Message Heikki Linnakangas 2013-11-15 18:08:30 Re: Sequence Access Method WIP