public inbox for [email protected]  
help / color / mirror / Atom feed
From: Lakshmi N <[email protected]>
To: [email protected]
Subject: [PATCH] Fix missing pfree(flags.data) in overexplain_debug
Date: Mon, 13 Apr 2026 13:33:11 -0700
Message-ID: <CA+3i_M9_R4t62NPqoa0pjkffMFzLeU30xGdBV4C-PUGHGh5W=w@mail.gmail.com> (raw)

Hi hackers,

attached a small patch to fix missing  pfree(flags.data) in
overexplain_debug.

Regards,
Lakshmi


Attachments:

  [application/octet-stream] 0001-Fix-missing-pfree-flags.data-in-overexplain_debug.patch (850B, 3-0001-Fix-missing-pfree-flags.data-in-overexplain_debug.patch)
  download | inline diff:
From 75d9ab6b6c5ed4fa21604766e28d84ec8a4b622e Mon Sep 17 00:00:00 2001
From: Lakshmi N <[email protected]>
Date: Mon, 13 Apr 2026 12:57:50 -0700
Subject: [PATCH] Fix missing pfree(flags.data) in overexplain_debug

---
 contrib/pg_overexplain/pg_overexplain.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/contrib/pg_overexplain/pg_overexplain.c b/contrib/pg_overexplain/pg_overexplain.c
index 715eda8dc56..d8fc1289cd5 100644
--- a/contrib/pg_overexplain/pg_overexplain.c
+++ b/contrib/pg_overexplain/pg_overexplain.c
@@ -405,6 +405,7 @@ overexplain_debug(PlannedStmt *plannedstmt, ExplainState *es)
 	if (flags.len == 0)
 		appendStringInfoString(&flags, ", none");
 	ExplainPropertyText("Flags", flags.data + 2, es);
+	pfree(flags.data);
 
 	/* Various lists of integers. */
 	overexplain_bitmapset("Subplans Needing Rewind",
-- 
2.43.0



view thread (2+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: [PATCH] Fix missing pfree(flags.data) in overexplain_debug
  In-Reply-To: <CA+3i_M9_R4t62NPqoa0pjkffMFzLeU30xGdBV4C-PUGHGh5W=w@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox