Fix ancient memory leak in PQprintTuples(); our code no longer uses this
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 19 Apr 2006 16:15:29 +0000 (16:15 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 19 Apr 2006 16:15:29 +0000 (16:15 +0000)
routine, but perhaps some applications do.  Found by Martijn van Oosterhout
using Coverity.

src/interfaces/libpq/fe-print.c

index f57d77bc91c21f8525023f25bd8f71f110123002..7a1e032fb7fadd7f352702eb984f8a0dea1de849 100644 (file)
@@ -10,7 +10,7 @@
  * didn't really belong there.
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/interfaces/libpq/fe-print.c,v 1.67 2006/03/05 15:59:09 momjian Exp $
+ *   $PostgreSQL: pgsql/src/interfaces/libpq/fe-print.c,v 1.68 2006/04/19 16:15:29 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -751,8 +751,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 */