On 2016-04-28 00:15, Tom Lane wrote:
Thomas Munro <thomas.mu...@enterprisedb.com> writes:
Also happens on OpenBSD 5.8.  Isn't this a classic case where memmove
is called for?  Replacing the memcpy at line 617 with memmove makes
the tests run successfully, but at first glance the other two
instances of memcpy in run_permutation should also be changed to
memmove, no?


Yeah, that's clearly busted.  Surprising that it has not failed on
any other platforms.

Suggested patch attached.

Will push in a moment.

And that seems to have done the trick. I started a manual run of HEAD on curculio and now it's green on the build farm.

Thanks!

/Mikael


--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to