From 6475e663b3c67895aa76acac27fd931bf8a6c0f1 Mon Sep 17 00:00:00 2001 From: Nathan Bossart Date: Tue, 29 Aug 2023 18:32:38 -0700 Subject: [PATCH] Fix misuse of PqMsg_Close. EndCommand() and EndReplicationCommand() should use PqMsg_CommandComplete instead. Oversight in commit f4b54e1ed9. Reported-by: Pavel Stehule, Tatsuo Ishii Author: Pavel Stehule Reviewed-by: Aleksander Alekseev, Michael Paquier Discussion: https://postgr.es/m/CAFj8pRAMDCJXjnwiCkCB1yO1f7NPggFY8PwwAJDnugu-Z2G-Cg%40mail.gmail.com --- src/backend/tcop/dest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/backend/tcop/dest.c b/src/backend/tcop/dest.c index 06d1872b9ac..bc8494ee7d4 100644 --- a/src/backend/tcop/dest.c +++ b/src/backend/tcop/dest.c @@ -176,7 +176,7 @@ EndCommand(const QueryCompletion *qc, CommandDest dest, bool force_undecorated_o len = BuildQueryCompletionString(completionTag, qc, force_undecorated_output); - pq_putmessage(PqMsg_Close, completionTag, len + 1); + pq_putmessage(PqMsg_CommandComplete, completionTag, len + 1); case DestNone: case DestDebug: @@ -200,7 +200,7 @@ EndCommand(const QueryCompletion *qc, CommandDest dest, bool force_undecorated_o void EndReplicationCommand(const char *commandTag) { - pq_putmessage(PqMsg_Close, commandTag, strlen(commandTag) + 1); + pq_putmessage(PqMsg_CommandComplete, commandTag, strlen(commandTag) + 1); } /* ----------------