Skip to content

Commit 55d1398

Browse files
committed
Stop OraParam walker from recursing into scalar fields
Follow-up to eb51c8e: after introducing WALK(), a few call sites still passed non-Node scalar members (extname, jointype, ctename) through the walker interface. Those are not parse-tree nodes and should not participate in recursive traversal. Drop those WALK() calls so recursion only follows real Node/List subtrees while preserving the OraParam numbering behavior for object and query structure fields.
1 parent eb51c8e commit 55d1398

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

src/backend/parser/parse_param.c

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1051,8 +1051,6 @@ raw_calculate_oraparamnumbers_walker(Node *node,
10511051
{
10521052
AlterExtensionContentsStmt *stmt = (AlterExtensionContentsStmt *)node;
10531053

1054-
if (WALK(stmt->extname))
1055-
return true;
10561054
if (WALK(stmt->object))
10571055
return true;
10581056
}
@@ -1570,16 +1568,12 @@ raw_calculate_oraparamnumbers_walker(Node *node,
15701568
return true;
15711569
if (WALK(stmt->subquery))
15721570
return true;
1573-
if (WALK(stmt->jointype))
1574-
return true;
15751571
if (WALK(stmt->joinaliasvars))
15761572
return true;
15771573
if (WALK(stmt->functions))
15781574
return true;
15791575
if (WALK(stmt->values_lists))
15801576
return true;
1581-
if (WALK(stmt->ctename))
1582-
return true;
15831577
if (WALK(stmt->coltypes))
15841578
return true;
15851579
if (WALK(stmt->coltypmods))

0 commit comments

Comments
 (0)