diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-06-16 12:17:03 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-06-16 12:17:38 -0400 |
| commit | 0b0baf26211a98a8593279fa24635bc4dd9ac99d (patch) | |
| tree | e3b48e3d8cad3594cd86db96307561cde6321f6b /src | |
| parent | b000afea6596c4dab2f0595ded751659a158b754 (diff) | |
| download | postgresql-0b0baf26211a98a8593279fa24635bc4dd9ac99d.tar.gz postgresql-0b0baf26211a98a8593279fa24635bc4dd9ac99d.zip | |
Avoid crash in "postgres -C guc" for a GUC with a null string value.
Emit "(null)" instead, which was the behavior all along on platforms
that don't crash, eg OS X. Per report from Jehan-Guillaume de Rorthais.
Back-patch to 9.2 where -C option was introduced.
Michael Paquier
Report: <20160615204036.2d35d86a@firost>
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/postmaster/postmaster.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c index 6421c8601bc..e350dd29252 100644 --- a/src/backend/postmaster/postmaster.c +++ b/src/backend/postmaster/postmaster.c @@ -827,10 +827,14 @@ PostmasterMain(int argc, char *argv[]) if (output_config_variable != NULL) { /* - * permission is handled because the user is reading inside the data - * dir + * "-C guc" was specified, so print GUC's value and exit. No extra + * permission check is needed because the user is reading inside the + * data dir. */ - puts(GetConfigOption(output_config_variable, false, false)); + const char *config_val = GetConfigOption(output_config_variable, + false, false); + + puts(config_val ? config_val : "(null)"); ExitPostmaster(0); } |