]> git.ipfire.org Git - thirdparty/postgresql.git/commit
In transformRowExpr(), check for too many columns in the row.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 29 Jul 2022 17:30:50 +0000 (13:30 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 29 Jul 2022 17:30:50 +0000 (13:30 -0400)
commitba2002d02cf4d6790d82d3b9ffd5d7a74a1c9589
treea3953fa8f609d2e84b85e56aa66f964d93b1c298
parent7cfe688dee86c10ed2622872f5b2f8ca9b1b7c5b
In transformRowExpr(), check for too many columns in the row.

A RowExpr with more than MaxTupleAttributeNumber columns would fail at
execution anyway, since we cannot form a tuple datum with more than that
many columns.  While heap_form_tuple() has a check for too many columns,
it emerges that there are some intermediate bits of code that don't
check and can be driven to failure with sufficiently many columns.
Checking this at parse time seems like the most appropriate place to
install a defense, since we already check SELECT list length there.

While at it, make the SELECT-list-length error use the same errcode
(TOO_MANY_COLUMNS) as heap_form_tuple does, rather than the generic
PROGRAM_LIMIT_EXCEEDED.

Per bug #17561 from Egor Chindyaskin.  The given test case crashes
in all supported branches (and probably a lot further back),
so patch all.

Discussion: https://postgr.es/m/17561-80350151b9ad2ad4@postgresql.org
src/backend/parser/parse_expr.c
src/backend/parser/parse_node.c