diff options
| author | Stephen Frost <sfrost@snowman.net> | 2013-07-14 14:35:26 -0400 |
|---|---|---|
| committer | Stephen Frost <sfrost@snowman.net> | 2013-07-14 14:35:26 -0400 |
| commit | 234e4cf6e1eac2f0e514379a2a533ffb71b33732 (patch) | |
| tree | 172ff3db683d5632a2daca20bb42e42e348c570f /src | |
| parent | 070518ddab2c94afea119f2b1944c05d16792b07 (diff) | |
| download | postgresql-234e4cf6e1eac2f0e514379a2a533ffb71b33732.tar.gz postgresql-234e4cf6e1eac2f0e514379a2a533ffb71b33732.zip | |
During parallel pg_dump, free commands from master
The command strings read by the child processes during parallel
pg_dump, after being read and handled, were not being free'd.
This patch corrects this relatively minor memory leak.
Leak found by the Coverity scanner.
Back patch to 9.3 where parallel pg_dump was introduced.
Diffstat (limited to 'src')
| -rw-r--r-- | src/bin/pg_dump/parallel.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/bin/pg_dump/parallel.c b/src/bin/pg_dump/parallel.c index ceab58b157c..7208b0fec23 100644 --- a/src/bin/pg_dump/parallel.c +++ b/src/bin/pg_dump/parallel.c @@ -922,6 +922,9 @@ WaitForCommands(ArchiveHandle *AH, int pipefd[2]) exit_horribly(modulename, "unrecognized command on communication channel: %s\n", command); + + /* command was pg_malloc'd and we are responsible for free()ing it. */ + free(command); } } |