Fix another oversight in logging of changes in postgresql.conf settings.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 8 Jul 2011 21:03:06 +0000 (17:03 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 8 Jul 2011 21:03:06 +0000 (17:03 -0400)
commit7c884433992cce9f53d87ca3a5d0d629e21af300
treeb1d387c45eed32762a25aa9d36f096321698b72a
parentec194651b8ba36e9ad5f7c98ac77c4d9639e2351
Fix another oversight in logging of changes in postgresql.conf settings.

We were using GetConfigOption to collect the old value of each setting,
overlooking the possibility that it didn't exist yet.  This does happen
in the case of adding a new entry within a custom variable class, as
exhibited in bug #6097 from Maxim Boguk.

To fix, add a missing_ok parameter to GetConfigOption, but only in 9.1
and HEAD --- it seems possible that some third-party code is using that
function, so changing its API in a minor release would cause problems.
In 9.0, create a near-duplicate function instead.
src/backend/commands/extension.c
src/backend/utils/misc/guc-file.l
src/backend/utils/misc/guc.c
src/include/utils/guc.h