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:38:43 -0400 |
commit | f5acde9380e164eab10b2dc3281bb1c07f690803 (patch) | |
tree | f776d26ba9cee48968254651f7c78d4a99d80705 /src/backend/regex/regexec.c | |
parent | 8839e7362c68470f8db66acdfa60b95a1c5312cf (diff) | |
download | postgresql-f5acde9380e164eab10b2dc3281bb1c07f690803.tar.gz postgresql-f5acde9380e164eab10b2dc3281bb1c07f690803.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/regex/regexec.c')
0 files changed, 0 insertions, 0 deletions