On Wed, Apr 07, 2021 at 09:55:40PM +0200, Peter Eisentraut wrote:
> Committed. Thanks!
I get a NULL pointer dereference if the function body has a doubled semicolon:
create function f() returns int language sql begin atomic select 1;; end;
Program received signal SIGSEGV, Segmentation fault.
transformStmt (pstate=pstate@entry=0x2623978, parseTree=parseTree@entry=0x0) at
analyze.c:297
297 switch (nodeTag(parseTree))
#0 transformStmt (pstate=pstate@entry=0x2623978,
parseTree=parseTree@entry=0x0) at analyze.c:297
#1 0x00000000006132a4 in interpret_AS_clause (queryString=<optimized out>,
sql_body_out=<synthetic pointer>, probin_str_p=<synthetic pointer>,
prosrc_str_p=<synthetic pointer>, inParameterNames=<optimized out>,
parameterTypes=<optimized out>,
sql_body_in=<optimized out>, as=<optimized out>, funcname=<optimized out>,
languageName=<optimized out>, languageOid=14) at functioncmds.c:937
#2 CreateFunction (pstate=pstate@entry=0x26213e0, stmt=stmt@entry=0x25fd048)
at functioncmds.c:1227
#3 0x0000000000813e23 in ProcessUtilitySlow (pstate=pstate@entry=0x26213e0,
pstmt=pstmt@entry=0x25fd3b8, queryString=queryString@entry=0x25fc040 "create
function f() returns int language sql begin atomic select 1;; end;",
context=context@entry=PROCESS_UTILITY_TOPLEVEL, params=params@entry=0x0,
queryEnv=queryEnv@entry=0x0, qc=qc@entry=0x7fff4b715b70, dest=0x25fd4a8) at
utility.c:1607
#4 0x0000000000812944 in standard_ProcessUtility (pstmt=0x25fd3b8,
queryString=0x25fc040 "create function f() returns int language sql begin
atomic select 1;; end;", context=PROCESS_UTILITY_TOPLEVEL, params=0x0,
queryEnv=0x0, dest=0x25fd4a8,
qc=0x7fff4b715b70) at utility.c:1034
#5 0x0000000000810efe in PortalRunUtility (portal=portal@entry=0x265fb60,
pstmt=0x25fd3b8, isTopLevel=isTopLevel@entry=true,
setHoldSnapshot=setHoldSnapshot@entry=false, dest=0x25fd4a8, qc=0x7fff4b715b70)
at pquery.c:1147
#6 0x0000000000811053 in PortalRunMulti (portal=portal@entry=0x265fb60,
isTopLevel=isTopLevel@entry=true, setHoldSnapshot=setHoldSnapshot@entry=false,
dest=dest@entry=0x25fd4a8, altdest=altdest@entry=0x25fd4a8,
qc=qc@entry=0x7fff4b715b70) at pquery.c:1310
#7 0x00000000008115e4 in PortalRun (portal=portal@entry=0x265fb60,
count=count@entry=9223372036854775807, isTopLevel=isTopLevel@entry=true,
run_once=run_once@entry=true, dest=dest@entry=0x25fd4a8,
altdest=altdest@entry=0x25fd4a8, qc=qc@entry=0x7fff4b715b70)
at pquery.c:786
#8 0x000000000080d004 in exec_simple_query (query_string=0x25fc040 "create
function f() returns int language sql begin atomic select 1;; end;") at
postgres.c:1214
#9 0x000000000080ee1f in PostgresMain (argc=argc@entry=1,
argv=argv@entry=0x7fff4b716030, dbname=0x2627788 "test", username=<optimized
out>) at postgres.c:4486
#10 0x000000000048bc97 in BackendRun (port=<optimized out>, port=<optimized
out>) at postmaster.c:4507
#11 BackendStartup (port=0x261f480) at postmaster.c:4229
#12 ServerLoop () at postmaster.c:1745
#13 0x000000000077c278 in PostmasterMain (argc=argc@entry=1,
argv=argv@entry=0x25f6a00) at postmaster.c:1417
#14 0x000000000048d51e in main (argc=1, argv=0x25f6a00) at main.c:209
(gdb) p parseTree
$1 = (Node *) 0x0