diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2006-04-19 16:15:52 +0000 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2006-04-19 16:15:52 +0000 |
commit | 8a3789cdfaddd1d1b01498baa0837292b898c45f (patch) | |
tree | c3fdf37d62268d491bff9b0c60208c8977615441 /src | |
parent | e1b40c521ee497836a28ef80e599668b8619d003 (diff) | |
download | postgresql-8a3789cdfaddd1d1b01498baa0837292b898c45f.tar.gz postgresql-8a3789cdfaddd1d1b01498baa0837292b898c45f.zip |
Fix ancient memory leak in PQprintTuples(); our code no longer uses this
routine, but perhaps some applications do. Found by Martijn van Oosterhout
using Coverity.
Diffstat (limited to 'src')
-rw-r--r-- | src/interfaces/libpq/fe-print.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/interfaces/libpq/fe-print.c b/src/interfaces/libpq/fe-print.c index d8831eb26d7..9a2eab519b9 100644 --- a/src/interfaces/libpq/fe-print.c +++ b/src/interfaces/libpq/fe-print.c @@ -10,7 +10,7 @@ * didn't really belong there. * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-print.c,v 1.48 2003/08/04 02:40:20 momjian Exp $ + * $Header: /cvsroot/pgsql/src/interfaces/libpq/fe-print.c,v 1.48.4.1 2006/04/19 16:15:52 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -729,8 +729,10 @@ PQprintTuples(const PGresult *res, fprintf(fout, "|\n%s\n", tborder); } } -} + if (tborder) + free(tborder); +} /* simply send out max-length number of filler characters to fp */ |