Prevent buffer overrun in read_tablespace_map().
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 17 Mar 2021 20:10:37 +0000 (16:10 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 17 Mar 2021 20:10:37 +0000 (16:10 -0400)
commita50e4fd028a1ece2b1a04df2c9ae6581783e9eef
treed6a0a97c3394d1651b7ee67b84f5c4b3c4f53d79
parent081876d75ea15c3bd2ee5ba64a794fd8ea46d794
Prevent buffer overrun in read_tablespace_map().

Robert Foggia of Trustwave reported that read_tablespace_map()
fails to prevent an overrun of its on-stack input buffer.
Since the tablespace map file is presumed trustworthy, this does
not seem like an interesting security vulnerability, but still
we should fix it just in the name of robustness.

While here, document that pg_basebackup's --tablespace-mapping option
doesn't work with tar-format output, because it doesn't.  To make it
work, we'd have to modify the tablespace_map file within the tarball
sent by the server, which might be possible but I'm not volunteering.
(Less-painful solutions would require changing the basebackup protocol
so that the source server could adjust the map.  That's not very
appetizing either.)
doc/src/sgml/ref/pg_basebackup.sgml
src/backend/access/transam/xlog.c