From db93e739ac61332126207b16f14da93f8ecac594 Mon Sep 17 00:00:00 2001
From: Alexander Korotkov <akorotkov@postgresql.org>
Date: Thu, 27 Apr 2023 22:06:14 +0300
Subject: [PATCH] Fix wrong construct_array_builtin() call in GUCArrayDelete()

The current code unintentionally uses the wrong datum to construct an array.
The bug was introduced by 096dd80f3c, so no backpatching is needed.

Reported-by: David Steele
Discussion: https://postgr.es/m/d46f9265-ff3c-6743-2278-6772598233c2%40pgmasters.net
Author: Nathan Bossart
Reviewed-by: David Steele, Tom Lane
---
 src/backend/utils/misc/guc.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c
index 9dd624b3ae0..53d1d9a06a7 100644
--- a/src/backend/utils/misc/guc.c
+++ b/src/backend/utils/misc/guc.c
@@ -6496,7 +6496,8 @@ GUCArrayDelete(ArrayType *array, ArrayType **usersetArray, const char *name)
 		{
 			newarray = construct_array_builtin(&d, 1, TEXTOID);
 			if (usersetArray)
-				newUsersetArray = construct_array_builtin(&d, 1, BOOLOID);
+				newUsersetArray = construct_array_builtin(&userSetDatum, 1,
+														  BOOLOID);
 		}
 
 		index++;