]> granicus.if.org Git - postgresql/commit
Fix failure to copy IndexScan.indexorderbyops in copyfuncs.c.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 18 May 2015 01:22:12 +0000 (21:22 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 18 May 2015 01:22:12 +0000 (21:22 -0400)
commit424661913c06af76a46fdff9cc24cc57abf14fb3
tree5d27c01f805aaf3cce92844344e51f37670573a4
parentb14cf229f4bd7238be2e31d873dc5dd241d3871e
Fix failure to copy IndexScan.indexorderbyops in copyfuncs.c.

This oversight results in a crash at executor startup if the plan has
been copied.  outfuncs.c was missed as well.

While we could probably have taught both those files to cope with the
originally chosen representation of an Oid array, it would have been
painful, not least because there'd be no easy way to verify the array
length.  An Oid List is far easier to work with.  And AFAICS, there is
no particular notational benefit to using an array rather than a list
in the existing parts of the patch either.  So just change it to a list.

Error in commit 35fcb1b3d038a501f3f4c87c05630095abaaadab, which is new,
so no need for back-patch.
src/backend/executor/nodeIndexscan.c
src/backend/nodes/copyfuncs.c
src/backend/nodes/outfuncs.c
src/backend/optimizer/plan/createplan.c
src/include/nodes/plannodes.h