[COMMITTERS] pgsql: Sanitize newlines in object names in "pg_restore -l" output. - Mailing list pgsql-committers

From Tom Lane
Subject [COMMITTERS] pgsql: Sanitize newlines in object names in "pg_restore -l" output.
Date
Msg-id E1cmQ0h-0002YY-7N@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Sanitize newlines in object names in "pg_restore -l" output.

Commits 89e0bac86 et al replaced newlines with spaces in object names
printed in SQL comments, but we neglected to consider that the same
names are also printed by "pg_restore -l", and a newline would render
the output unparseable by "pg_restore -L".  Apply the same replacement
in "-l" output.  Since "pg_restore -L" doesn't actually examine any
object names, only the dump ID field that starts each line, this is
enough to fix things for its purposes.

The previous fix was treated as a security issue, and we might have
done that here as well, except that the issue was reported publicly
to start with.  Anyway it's hard to see how this could be exploited
for SQL injection; "pg_restore -L" doesn't do much with the file
except parse it for leading integers.

Per bug #14587 from Milos Urbanek.  Back-patch to all supported versions.

Discussion: https://postgr.es/m/20170310155318.1425.30483@wrigleys.postgresql.org

Branch
------
REL9_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/0c0a95c2f83b4ed88ebb7f7759c22cd73e7b3cf3

Modified Files
--------------
src/bin/pg_dump/pg_backup_archiver.c | 39 +++++++++++++++++++++++++++++++-----
1 file changed, 34 insertions(+), 5 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: [COMMITTERS] pgsql: Change the relkind for partitioned tables from 'P' to 'p'.
Next
From: Andres Freund
Date:
Subject: [COMMITTERS] pgsql: Enable 64 bit atomics on ARM64.