Fix a memory leak in tuplestore_end(). Unlikely to be significant during

Neil Conway · Neil Conway · commit 494d6f809eb6 · 2007-08-02T17:48:52.000Z
normal operation, but tuplestore_end() ought to do what it claims to do.
diff --git a/src/backend/utils/sort/tuplestore.c b/src/backend/utils/sort/tuplestore.c
@@ -38,7 +38,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/utils/sort/tuplestore.c,v 1.33 2007/06/07 19:19:57 tgl Exp $
+ *	  $PostgreSQL: pgsql/src/backend/utils/sort/tuplestore.c,v 1.34 2007/08/02 17:48:52 neilc Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -322,6 +322,7 @@ tuplestore_end(Tuplestorestate *state)
 			pfree(state->memtuples[i]);
 		pfree(state->memtuples);
 	}
+	pfree(state);
 }
 
 /*

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@`
`38`	`38`	`* Portions Copyright (c) 1994, Regents of the University of California`
`39`	`39`	`*`
`40`	`40`	`* IDENTIFICATION`
`41`		`- * $PostgreSQL: pgsql/src/backend/utils/sort/tuplestore.c,v 1.33 2007/06/07 19:19:57 tgl Exp $`
	`41`	`+ * $PostgreSQL: pgsql/src/backend/utils/sort/tuplestore.c,v 1.34 2007/08/02 17:48:52 neilc Exp $`
`42`	`42`	`*`
`43`	`43`	`*-------------------------------------------------------------------------`
`44`	`44`	`*/`
`@@ -322,6 +322,7 @@ tuplestore_end(Tuplestorestate *state)`
`322`	`322`	`pfree(state->memtuples[i]);`
`323`	`323`	`pfree(state->memtuples);`
`324`	`324`	`}`
	`325`	`+ pfree(state);`
`325`	`326`	`}`
`326`	`327`
`327`	`328`	`/*`