fix conversion of varchar to binary varbinary and vice versa (babelfi…

…sh-for-postgresql#248) Release syscache reference when returning not null nodes from string literal hook to avoid cache reference leak Engine PR: babelfish-for-postgresql#248 Extension PR: babelfish-for-postgresql/babelfish_extensions#1957 Task: 1940 Signed-off-by: Tanzeel Khan <[email protected]>
amazon-aurora · Nov 17, 2023 · 5dad5ed · 5dad5ed
1 parent 19c6eca
commit 5dad5ed
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/backend/parser/parse_coerce.c b/src/backend/parser/parse_coerce.c
@@ -334,9 +334,11 @@ coerce_type(ParseState *pstate, Node *node,
 												  targetTypeMod, baseTypeMod,
 												  newcon, DatumGetCString(con->constvalue),
 												  ccontext, cformat, location);
-			if (result) /* runtimer error function returned */
+			if (result)
+			{
+				ReleaseSysCache(baseType);
 				return result;
-
+			}
 			/*
 			 * If the result is NULL, the newcon should be already updated properly.
 			 * Keep advancing the code.