]> granicus.if.org Git - postgresql/commit
Reject empty names and recursion in config-file include directives.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 27 Aug 2019 18:44:26 +0000 (14:44 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 27 Aug 2019 18:44:26 +0000 (14:44 -0400)
commit6e42130568ad28bed857948f5a3192dbf01624dc
treede40f9b40464422e5a0ba431dac650e2bf1ae667
parent9acda731184c1ebdf99172cbb19d0404b7eebc37
Reject empty names and recursion in config-file include directives.

An empty file name or subdirectory name leads join_path_components() to
just produce the parent directory name, which leads to weird failures or
recursive inclusions.  Let's throw a specific error for that.  It takes
only slightly more code to detect all-blank names, so do so.

Also, detect direct recursion, ie a file calling itself.  As coded
this will also detect recursion via "include_dir '.'", which is
perhaps more likely than explicitly including the file itself.

Detecting indirect recursion would require API changes for guc-file.l
functions, which seems not worth it since extensions might call them.
The nesting depth limit will catch such cases eventually, just not
with such an on-point error message.

In passing, adjust the example usages in postgresql.conf.sample
to perhaps eliminate the problem at the source: there's no reason
for the examples to suggest that an empty value is valid.

Per a trouble report from Brent Bates.  Back-patch to 9.5; the
issue is old, but the code in 9.4 is enough different that the
patch doesn't apply easily, and it doesn't seem worth the trouble
to fix there.

Ian Barwick and Tom Lane

Discussion: https://postgr.es/m/8c8bcbca-3bd9-dc6e-8986-04a5abdef142@2ndquadrant.com
src/backend/utils/misc/guc-file.l
src/backend/utils/misc/postgresql.conf.sample