Fix cursor quoting in FetchStmt (#267)

* Fix cursor quoting in FetchStmt Regression tests updated to test all cursor related statement with a name that requires quoting. * Test quoting in `deparse_tests.c` --------- Co-authored-by: msepga <[email protected]>
pganalyze · Oct 31, 2024 · fce106a · fce106a
1 parent 0667029
commit fce106a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/postgres_deparse.c b/src/postgres_deparse.c
@@ -8617,7 +8617,7 @@ static void deparseFetchStmt(StringInfo str, FetchStmt *fetch_stmt)
 			appendStringInfo(str, "RELATIVE %ld ", fetch_stmt->howMany);
 	}
 
-	appendStringInfoString(str, fetch_stmt->portalname);
+	appendStringInfoString(str, quote_identifier(fetch_stmt->portalname));
 }
 
 static void deparseAlterDefaultPrivilegesStmt(StringInfo str, AlterDefaultPrivilegesStmt *alter_default_privileges_stmt)

diff --git a/test/deparse_tests.c b/test/deparse_tests.c
@@ -407,7 +407,9 @@ const char* tests[] = {
   "ALTER TABLE a DISABLE TRIGGER ALL",
   "ALTER TABLE a DISABLE TRIGGER USER",
   "CREATE INDEX myindex ON public.mytable USING btree (col1, (col2::varchar) varchar_pattern_ops)",
-  "SELECT * FROM CAST(1 AS text)"
+  "SELECT * FROM CAST(1 AS text)",
+  "DECLARE \"Foo1\" SCROLL CURSOR FOR SELECT * FROM tenk1 ORDER BY unique2",
+  "FETCH BACKWARD 23 \"Foo1\""
 };
 
 size_t testsLength = __LINE__ - 4;