Fix minor memory leak in ident_inet().
authorTom Lane
Thu, 12 Feb 2015 00:09:54 +0000 (19:09 -0500)
committerTom Lane
Thu, 12 Feb 2015 00:09:54 +0000 (19:09 -0500)
commite44735c48821e40b2679d5b69c4d47e98ddaae6f
treee9a10a03b39001962966584aa082619b77bd1dd3
parentee8d3778311e08d810f79c97f90ffb678063a59f
Fix minor memory leak in ident_inet().

We'd leak the ident_serv data structure if the second pg_getaddrinfo_all
(the one for the local address) failed.  This is not of great consequence
because a failure return here just leads directly to backend exit(), but
if this function is going to try to clean up after itself at all, it should
not have such holes in the logic.  Try to fix it in a future-proof way by
having all the failure exits go through the same cleanup path, rather than
"optimizing" some of them.

Per Coverity.  Back-patch to 9.2, which is as far back as this patch
applies cleanly.
src/backend/libpq/auth.c