NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit225e1dd

committed

Release temporary array in check_for_data_types_usage().

Coverity identified this as a resource leak. It's surely of noconsequence given that the function is called only once per run, butfreeing the storage is no more work than dismissing the complaint.Minor oversight in commit347758b.

1 parentfc2d260 commit225e1ddCopy full SHA for 225e1dd

File tree

1 file changed

-0

lines changed

src/bin/pg_upgrade
- check.c

1 file changed

-0

lines changed

`‎src/bin/pg_upgrade/check.c`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -521,6 +521,8 @@ check_for_data_types_usage(ClusterInfo cluster, DataTypesUsageChecks checks)`
`521`	`521`	`if (found)`
`522`	`522`	`pg_fatal("Data type checks failed: %s",report.data);`
`523`	`523`
	`524`	`+pg_free(results);`
	`525`	`+`
`524`	`526`	`check_ok();`
`525`	`527`	`}`
`526`	`528`

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!

Commit225e1dd

File tree

1 file changed

1 file changed

`‎src/bin/pg_upgrade/check.c`

0 commit comments