pgsql: Fix minor memory leak in ident_inet(). - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix minor memory leak in ident_inet().
Date
Msg-id E1YLhM6-0002H7-Rq@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
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.

Branch
------
REL9_2_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/0168fb07dad578433c5819bcaa3fc71a5a9dc0eb

Modified Files
--------------
src/backend/libpq/auth.c |   21 +++++++++++----------
1 file changed, 11 insertions(+), 10 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix minor memory leak in ident_inet().
Next
From: Tom Lane
Date:
Subject: pgsql: Fix minor memory leak in ident_inet().