NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitaf0f200

committed

Fix pg_size_pretty() to avoid overflow for inputs close to INT64_MAX.

The expression that tried to round the value to the nearest TB couldoverflow, leading to bogus output as reported in bug #5993 from NicolaCossu. This isn't likely to ever happen in the intended usage of thefunction (if it could, we'd be needing to use a wider datatype instead);but it's not hard to give the expected output, so let's do so.

1 parentf8ebe3b commitaf0f200Copy full SHA for af0f200

File tree

1 file changed

-1

lines changed

src/backend/utils/adt
- dbsize.c

1 file changed

-1

lines changed

`‎src/backend/utils/adt/dbsize.c`

Lines changed: 7 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -490,9 +490,15 @@ pg_size_pretty(PG_FUNCTION_ARGS)`
`490`	`490`	`(size+mult /2) /mult);`
`491`	`491`	`else`
`492`	`492`	`{`
	`493`	`+/* Here we have to worry about avoiding overflow */`
	`494`	`+int64val;`
	`495`	`+`
`493`	`496`	`mult *=1024;`
	`497`	`+val=size /mult;`
	`498`	`+if ((size %mult) >= (mult /2))`
	`499`	`+val++;`
`494`	`500`	`snprintf(buf,sizeof(buf),INT64_FORMAT" TB",`
`495`		`-(size+mult /2) /mult);`
	`501`	`+val);`
`496`	`502`	`}`
`497`	`503`	`}`
`498`	`504`	`}`

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!

Commitaf0f200

File tree

1 file changed

1 file changed

`‎src/backend/utils/adt/dbsize.c`

0 commit comments