Commit4a0a5f2

committed

vacuumlo: Avoid unlikely memory leak.

Spotted by Coverity. This isn't likely to matter in practice, butthere's no harm in fixing it.Michael Paquier

1 parent59f71a0 commit4a0a5f2Copy full SHA for 4a0a5f2

File tree

-0

lines changed

-0

lines changed

Lines changed: 9 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -240,6 +240,12 @@ vacuumlo(const char database, const struct _param param)`
`240`	`240`	`fprintf(stderr,"Out of memory\n");`
`241`	`241`	`PQclear(res);`
`242`	`242`	`PQfinish(conn);`
	`243`	`+if (schema!=NULL)`
	`244`	`+PQfreemem(schema);`
	`245`	`+if (schema!=NULL)`
	`246`	`+PQfreemem(table);`
	`247`	`+if (schema!=NULL)`
	`248`	`+PQfreemem(field);`
`243`	`249`	`return-1;`
`244`	`250`	`}`
`245`	`251`
`@@ -256,6 +262,9 @@ vacuumlo(const char database, const struct _param param)`
`256`	`262`	`PQclear(res2);`
`257`	`263`	`PQclear(res);`
`258`	`264`	`PQfinish(conn);`
	`265`	`+PQfreemem(schema);`
	`266`	`+PQfreemem(table);`
	`267`	`+PQfreemem(field);`
`259`	`268`	`return-1;`
`260`	`269`	`}`
`261`	`270`	`PQclear(res2);`

Comments

(0)