Feedback-ID: ia2694551:Fastmail
Date: Tue, 21 Jul 2026 15:07:33 +0200
From: Alvaro Herrera <alvherre@alvh.no-ip.org>
To: "Jonathan Gonzalez V." <jonathan.abdiel@gmail.com>
Cc: Chao Li <li.evan.chao@gmail.com>, Euler Taveira <euler@eulerto.com>,
	japin <japinli@hotmail.com>, Andres Freund <andres@anarazel.de>,
	pgsql-hackers@lists.postgresql.org
Subject: Re: log_min_messages per backend type
Message-ID: <al9UOWZfiwOioZJB@alvherre.pgsql>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <87bjc0heaq.fsf@gmail.com>
Archived-At: 
 <https://www.postgresql.org/message-id/al9UOWZfiwOioZJB%40alvherre.pgsql>
Precedence: bulk

On 2026-Jul-21, Jonathan Gonzalez V. wrote:

> but the
> second one is breaking the detection of the messages here:
> 
> @@ -2392,6 +2393,8 @@ check_log_min_messages(char **newval, void **extra, GucSource source)
>                 guc_free(rawstring);
>                 return false;
>         }
> +       rawstring_len = strlen(rawstring);
> +       guc_free(rawstring);
> 
> 
> The rawstring is being reference in the elemlist, so the free is making
> the string empty when is going to be used later:

Yeah, the second one is clearly busted.  Please consider it withdrawn.

-- 
Álvaro Herrera               48°01'N 7°57'E  —  https://www.EnterpriseDB.com/
"After a quick R of TFM, all I can say is HOLY CR** THAT IS COOL! PostgreSQL was
amazing when I first started using it at 7.2, and I'm continually astounded by
learning new features and techniques made available by the continuing work of
the development team."
Berend Tober, http://archives.postgresql.org/pgsql-hackers/2007-08/msg01009.php