NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit88ca787

committed

Fix copy-pasto in freeing memory on error in vacuumlo.

It's harmless to call PQfreemem() with a NULL argument, so the onlyconsequence was that if allocating 'schema' failed, but allocating 'table'or 'field' succeeded, we would leak a bit of memory. That's highlyunlikely to happen, so this is just academical, but let's get it right.Per bug #15838 from Timur Birsh. Backpatch back to 9.5, where thePQfreemem() calls were introduced.Discussion:https://www.postgresql.org/message-id/15838-3221652c72c5e69d@postgresql.org

1 parentfac1ed2 commit88ca787Copy full SHA for 88ca787

File tree

1 file changed

-2

lines changed

contrib/vacuumlo
- vacuumlo.c

1 file changed

-2

lines changed

`‎contrib/vacuumlo/vacuumlo.c‎`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -245,9 +245,9 @@ vacuumlo(const char database, const struct _param param)`
`245`	`245`	`PQfinish(conn);`
`246`	`246`	`if (schema!=NULL)`
`247`	`247`	`PQfreemem(schema);`
`248`		`-if (schema!=NULL)`
	`248`	`+if (table!=NULL)`
`249`	`249`	`PQfreemem(table);`
`250`		`-if (schema!=NULL)`
	`250`	`+if (field!=NULL)`
`251`	`251`	`PQfreemem(field);`
`252`	`252`	`return-1;`
`253`	`253`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit88ca787

File tree

1 file changed

1 file changed

`‎contrib/vacuumlo/vacuumlo.c‎`

0 commit comments