On 2016-04-28 00:15, Tom Lane wrote:
> Thomas Munro <thomas.munro@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
