Avoid formally-undefined use of memcpy() in hstoreUniquePairs().
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 25 Nov 2017 19:42:10 +0000 (14:42 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 25 Nov 2017 19:42:25 +0000 (14:42 -0500)
commitddba320059fcce0126ea43f0c0404d3ed1ce47aa
tree1e918f20d4e1d4d808f5ce6118ddedcdb71fd5c5
parent5dc7faa91e195a64f58057b7fb90778abfec065c
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