rdelaage
/
postgresql
mirror of https://git.postgresql.org/git/postgresql.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix assorted memory leaks in pg_hba.conf parsing. Over a sufficiently 

large number of SIGHUP cycles, these would have run the postmaster out
of memory.  Noted while testing memory-leak scenario in postgresql.conf
configuration-change-printing patch.
This commit is contained in:
Tom Lane 2009-10-03 20:04:39 +00:00
parent 54d60bbd07
commit 35a173ab33
1 changed files with 14 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
src/backend/libpq/hba.c
View File

@ -10,7 +10,7 @@
*
*
* IDENTIFICATION
* $PostgreSQL: pgsql/src/backend/libpq/hba.c,v 1.191 2009/10/01 01:58:57 tgl Exp $
* $PostgreSQL: pgsql/src/backend/libpq/hba.c,v 1.192 2009/10/03 20:04:39 tgl Exp $
*
*-------------------------------------------------------------------------
*/
@ -804,16 +804,12 @@ parse_hba_line(List *line, int line_num, HbaLine *parsedline)
token, gai_strerror(ret)),
errcontext("line %d of configuration file \"%s\"",
line_num, HbaFileName)));
if (cidr_slash)
*cidr_slash = '/';
if (gai_result)
pg_freeaddrinfo_all(hints.ai_family, gai_result);
pfree(token);
return false;
}
if (cidr_slash)
*cidr_slash = '/';
memcpy(&parsedline->addr, gai_result->ai_addr,
gai_result->ai_addrlen);
pg_freeaddrinfo_all(hints.ai_family, gai_result);
@ -824,18 +820,22 @@ parse_hba_line(List *line, int line_num, HbaLine *parsedline)
if (pg_sockaddr_cidr_mask(&parsedline->mask, cidr_slash + 1,
parsedline->addr.ss_family) < 0)
{
*cidr_slash = '/'; /* restore token for message */
ereport(LOG,
(errcode(ERRCODE_CONFIG_FILE_ERROR),
errmsg("invalid CIDR mask in address \"%s\"",
token),
errcontext("line %d of configuration file \"%s\"",
line_num, HbaFileName)));
pfree(token);
return false;
}
pfree(token);
}
else
{
/* Read the mask field. */
pfree(token);
line_item = lnext(line_item);
if (!line_item)
{
@ -1266,7 +1266,7 @@ check_hba(hbaPort *port)
}
/*
* Free the contents of a hba record
* Free an HbaLine structure
*/
static void
free_hba_record(HbaLine *record)
@ -1275,6 +1275,8 @@ free_hba_record(HbaLine *record)
pfree(record->database);
if (record->role)
pfree(record->role);
if (record->usermap)
pfree(record->usermap);
if (record->pamservice)
pfree(record->pamservice);
if (record->ldapserver)
@ -1287,6 +1289,7 @@ free_hba_record(HbaLine *record)
pfree(record->krb_server_hostname);
if (record->krb_realm)
pfree(record->krb_realm);
pfree(record);
}
/*
@ -1355,20 +1358,22 @@ load_hba(void)
{
/* Parse error in the file, so indicate there's a problem */
free_hba_record(newline);
pfree(newline);
ok = false;
/*
* Keep parsing the rest of the file so we can report errors on
* more than the first row. Error has already been reported in the
* parsing function, so no need to log it here.
*/
ok = false;
continue;
}
new_parsed_lines = lappend(new_parsed_lines, newline);
}
/* Free the temporary lists */
free_lines(&hba_lines, &hba_line_nums);
if (!ok)
{
/* Parsing failed at one or more rows, so bail out */
@ -1380,9 +1385,6 @@ load_hba(void)
clean_hba_list(parsed_hba_lines);
parsed_hba_lines = new_parsed_lines;
/* Free the temporary lists */
free_lines(&hba_lines, &hba_line_nums);
return true;
}