diff options
author | Stephen Frost <sfrost@snowman.net> | 2013-07-14 15:31:23 -0400 |
---|---|---|
committer | Stephen Frost <sfrost@snowman.net> | 2013-07-14 15:31:23 -0400 |
commit | d368a301b3a4bf5fec17e81c630adddeac80a7fc (patch) | |
tree | 99deb30d128b40387a3e96cef81700680588a633 /src/backend/executor/nodeModifyTable.c | |
parent | 234e4cf6e1eac2f0e514379a2a533ffb71b33732 (diff) | |
download | postgresql-d368a301b3a4bf5fec17e81c630adddeac80a7fc.tar.gz postgresql-d368a301b3a4bf5fec17e81c630adddeac80a7fc.zip |
pg_receivexlog - Exit on failure to parse
In streamutil.c:GetConnection(), upgrade failure to parse the
connection string to an exit(1) instead of simply returning NULL.
Most callers already immediately exited, but pg_receivexlog would
loop on this case, continually trying to re-parse the connection
string (which can't be changed after pg_receivexlog has started).
GetConnection() was already expected to exit(1) in some cases
(eg: failure to allocate memory or if unable to determine the
integer_datetimes flag), so this change shouldn't surprise anyone.
Began looking at this due to the Coverity scanner complaining that
we were leaking err_msg in this case- no longer an issue since we
just exit(1) immediately.
Diffstat (limited to 'src/backend/executor/nodeModifyTable.c')
0 files changed, 0 insertions, 0 deletions