]> granicus.if.org Git - postgresql/commit
Fix misuse of memcpy() in check_ip().
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 16 Feb 2015 21:17:48 +0000 (16:17 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 16 Feb 2015 21:18:08 +0000 (16:18 -0500)
commit3913b897d61975e2b2b7e0ac32b3b6f657eb2c5f
tree1d5d07ffbae28b245fa8334cb93089dce6108756
parenteffcaa4c286da355e7ee59a6e774c4cd7a0ea4f5
Fix misuse of memcpy() in check_ip().

The previous coding copied garbage into a local variable, pretty much
ensuring that the intended test of an IPv6 connection address against a
promoted IPv4 address from pg_hba.conf would never match.  The lack of
field complaints likely indicates that nobody realized this was supposed
to work, which is unsurprising considering that no user-facing docs suggest
it should work.

In principle this could have led to a SIGSEGV due to reading off the end of
memory, but since the source address would have pointed to somewhere in the
function's stack frame, that's quite unlikely.  What led to discovery of
the bug is Hugo Osvaldo Barrera's report of a crash after an OS upgrade,
which is probably because he is now running a system in which memcpy raises
abort() upon detecting overlapping source and destination areas.  (You'd
have to additionally suppose some things about the stack frame layout to
arrive at this conclusion, but it seems plausible.)

This has been broken since the code was added, in commit f3aec2c7f51904e7,
so back-patch to all supported branches.
src/backend/libpq/hba.c