NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitd5c3d6c

committed

Fix integer underflow in shared memory debugging

dsa_dump would print a large negative number instead of zero forsegment bin 0. Fix by explicitly checking for underflow and addspecial case for bin 0. Backpatch to all supported versions.Author: Ian Ilyasov <ianilyasov@outlook.com>Reviewed-by: Robert Haas <robertmhaas@gmail.com>Discussion:https://postgr.es/m/GV1P251MB1004E0D09D117D3CECF9256ECD502@GV1P251MB1004.EURP251.PROD.OUTLOOK.COMBackpatch-through: v12

1 parent43cca9d commitd5c3d6cCopy full SHA for d5c3d6c

File tree

1 file changed

-3

lines changed

src/backend/utils/mmgr
- dsa.c

1 file changed

-3

lines changed

`‎src/backend/utils/mmgr/dsa.c`

Lines changed: 7 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -1093,9 +1093,13 @@ dsa_dump(dsa_area *area)`
`1093`	`1093`	`{`
`1094`	`1094`	`dsa_segment_indexsegment_index;`
`1095`	`1095`
`1096`		`-fprintf(stderr,`
`1097`		`-" segment bin %zu (at least %d contiguous pages free):\n",`
`1098`		`-i,1 << (i-1));`
	`1096`	`+if (i==0)`
	`1097`	`+fprintf(stderr,`
	`1098`	`+" segment bin %zu (no contiguous free pages):\n",i);`
	`1099`	`+else`
	`1100`	`+fprintf(stderr,`
	`1101`	`+" segment bin %zu (at least %d contiguous pages free):\n",`
	`1102`	`+i,1 << (i-1));`
`1099`	`1103`	`segment_index=area->control->segment_bins[i];`
`1100`	`1104`	`while (segment_index!=DSA_SEGMENT_INDEX_NONE)`
`1101`	`1105`	`{`

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!

Commitd5c3d6c

File tree

1 file changed

1 file changed

`‎src/backend/utils/mmgr/dsa.c`

0 commit comments