NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit6ba0d8d

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 parent147d626 commit6ba0d8dCopy full SHA for 6ba0d8d

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
`@@ -497,9 +497,15 @@ pg_size_pretty(PG_FUNCTION_ARGS)`
`497`	`497`	`(size+mult /2) /mult);`
`498`	`498`	`else`
`499`	`499`	`{`
	`500`	`+/* Here we have to worry about avoiding overflow */`
	`501`	`+int64val;`
	`502`	`+`
`500`	`503`	`mult *=1024;`
	`504`	`+val=size /mult;`
	`505`	`+if ((size %mult) >= (mult /2))`
	`506`	`+val++;`
`501`	`507`	`snprintf(buf,sizeof(buf),INT64_FORMAT" TB",`
`502`		`-(size+mult /2) /mult);`
	`508`	`+val);`
`503`	`509`	`}`
`504`	`510`	`}`
`505`	`511`	`}`

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!

Commit6ba0d8d

File tree

1 file changed

1 file changed

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

0 commit comments