Re: ALTER SYSTEM SET command to change postgresql.conf parameters

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tatsuo Ishii <ishii(at)postgresql(dot)org>
Cc: amit(dot)kapila16(at)gmail(dot)com, haribabu(dot)kommi(at)huawei(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: ALTER SYSTEM SET command to change postgresql.conf parameters
Date: 2013-12-18 14:00:40
Message-ID: 52B1AA88.1000601@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 12/18/2013 03:35 AM, Tatsuo Ishii wrote:
> 3) initdb.c
>
> It seems the memory allocated for autoconflines[0] and
> autoconflines[1] by pg_strdup is never freed.
>
> (I think there's similar problem with "conflines" as well, though it
> was not introduced by the patch).
>

Why would we care? initdb doesn't run for very long, and the memory will
be freed when it exits, usually within a few seconds. My recollection
from back when I originally rewrote initdb in C was that cleaning up the
memory leaks wasn't worth the trouble.

cheers

andrew

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2013-12-18 14:03:20 Re: [PATCH] SQL assertions prototype
Previous Message Stephen Frost 2013-12-18 13:47:28 Re: stats for network traffic WIP