]> granicus.if.org Git - postgresql/commit
Check for stack overflow in transformSetOperationTree().
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 12 Nov 2012 00:56:16 +0000 (19:56 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 12 Nov 2012 00:56:16 +0000 (19:56 -0500)
commit454edf1da9f36615b780b1cb216ae5d9fe8ba42c
treea60af5a7bcaf153dfb60c8925ed1d4201dd0e99e
parent9b06d91ac88d2263d185fb12c7f11d9b450326db
Check for stack overflow in transformSetOperationTree().

Since transformSetOperationTree() recurses, it can be driven to stack
overflow with enough UNION/INTERSECT/EXCEPT clauses in a query.  Add a
check to ensure it fails cleanly instead of crashing.  Per report from
Matthew Gerber (though it's not clear whether this is the only thing
going wrong for him).

Historical note: I think the reasoning behind not putting a check here in
the beginning was that the check in transformExpr() ought to be sufficient
to guard the whole parser.  However, because transformSetOperationTree()
recurses all the way to the bottom of the set-operation tree before doing
any analysis of the statement's expressions, that check doesn't save it.
src/backend/parser/analyze.c