aboutsummaryrefslogtreecommitdiff
path: root/src/backend/access/gist/gistxlog.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2012-11-11 19:56:10 -0500
committerTom Lane <tgl@sss.pgh.pa.us>2012-11-11 19:56:10 -0500
commit34f3b396a6554903007031b0541781b7ecdd6e4d (patch)
tree6a5b2a4553f36f0ee2711db8cc57501cbbde64c0 /src/backend/access/gist/gistxlog.c
parentfa12cb7f0241c767a072c980d75780c9ec036e01 (diff)
downloadpostgresql-34f3b396a6554903007031b0541781b7ecdd6e4d.tar.gz
postgresql-34f3b396a6554903007031b0541781b7ecdd6e4d.zip
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.
Diffstat (limited to 'src/backend/access/gist/gistxlog.c')
0 files changed, 0 insertions, 0 deletions