Fix memleaks in json encoding using temporary memory context

Nikita Glukhov · Nikita Glukhov · commit c958d6aa6369 · 2022-07-27T01:02:30.000+03:00
diff --git a/src/backend/utils/adt/jsonb_util.c b/src/backend/utils/adt/jsonb_util.c
@@ -1742,12 +1742,29 @@ void *
 JsonEncode(const JsonbValue *val, JsonValueEncoder encoder, void *cxt)
 {
 	StringInfoData	buffer;
+#if 0
+	MemoryContext	tmpcxt,
+					oldcxt;
+#endif
 
 	/* Allocate an output buffer. It will be enlarged as needed */
 	initStringInfo(&buffer);
 
+#if 0
+	tmpcxt = AllocSetContextCreate(CurrentMemoryContext,
+								   "Json Encoding Context",
+								   ALLOCSET_DEFAULT_MINSIZE,
+								   ALLOCSET_DEFAULT_INITSIZE,
+								   ALLOCSET_DEFAULT_MAXSIZE);
+	oldcxt = MemoryContextSwitchTo(tmpcxt);
+#endif
+
 	(*encoder)(&buffer, val, cxt);
 
+#if 0
+	MemoryContextSwitchTo(oldcxt);
+	MemoryContextDelete(tmpcxt);
+#endif
 	/*
 	 * Note: the JEntry of the root is discarded. Therefore the root
 	 * JsonbContainer struct must contain enough information to tell what kind
