NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit6032991

committed

Fix possible leak of semaphore count.

Commit4aec498 reorganized the orderof operations here so that we no longer increment the number of "extrawaits" before locking the semaphore, but it did not change thestarting value of extraWaits from 0 to -1 to compensate. In the worstcase, this could leak a semaphore count, but that seems to be unlikelyin practice.Discussion:http://postgr.es/m/CAA4eK1JyVqXiMba+-a589Rk0pyHsyKkGxeumVKjU6Y74hdrVLQ@mail.gmail.comAmit Kapila, per an off-list report by Dilip Kumar. Reviewed by me.

1 parent5b4f8f4 commit6032991Copy full SHA for 6032991

File tree

1 file changed

-1

lines changed

src/backend/storage/ipc
- procarray.c

1 file changed

-1

lines changed

`‎src/backend/storage/ipc/procarray.c`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -491,7 +491,6 @@ ProcArrayGroupClearXid(PGPROC *proc, TransactionId latestXid)`
`491`	`491`	`volatilePROC_HDR*procglobal=ProcGlobal;`
`492`	`492`	`uint32nextidx;`
`493`	`493`	`uint32wakeidx;`
`494`		`-intextraWaits=-1;`
`495`	`494`
`496`	`495`	`/* We should definitely have an XID to clear. */`
`497`	`496`	`Assert(TransactionIdIsValid(allPgXact[proc->pgprocno].xid));`
`@@ -518,6 +517,8 @@ ProcArrayGroupClearXid(PGPROC *proc, TransactionId latestXid)`
`518`	`517`	`*/`
`519`	`518`	`if (nextidx!=INVALID_PGPROCNO)`
`520`	`519`	`{`
	`520`	`+intextraWaits=0;`
	`521`	`+`
`521`	`522`	`/* Sleep until the leader clears our XID. */`
`522`	`523`	`for (;;)`
`523`	`524`	`{`

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!

Commit6032991

File tree

1 file changed

1 file changed

`‎src/backend/storage/ipc/procarray.c`

0 commit comments