Avoid formally-undefined use of memcpy() in hstoreUniquePairs().
authorTom Lane
Sat, 25 Nov 2017 19:42:10 +0000 (14:42 -0500)
committerTom Lane
Sat, 25 Nov 2017 19:42:41 +0000 (14:42 -0500)
commit5c38ddebd66bc239b23fe20d489fe03c9ebd4e28
treeeb3e79d5889727a7f962b58d3a15f19ae6de58cf
parent2e105cf6db214ec69e62e4c4dfc71e39879485ae
Avoid formally-undefined use of memcpy() in hstoreUniquePairs().

hstoreUniquePairs() often called memcpy with equal source and destination
pointers.  Although this is almost surely harmless in practice, it's
undefined according to the letter of the C standard.  Some versions of
valgrind will complain about it, and some versions of libc as well
(cf. commit ad520ec4a).  Tweak the code to avoid doing that.

Noted by Tomas Vondra.  Back-patch to all supported versions because
of the hazard of libc assertions.

Discussion: https://postgr.es/m/bf84d940-90d4-de91-19dd-612e011007f4@fuzzy.cz
contrib/hstore/hstore_io.c