]> granicus.if.org Git - postgresql/commit
Fix possible leak of semaphore count.
authorRobert Haas <rhaas@postgresql.org>
Thu, 5 Jan 2017 18:12:16 +0000 (13:12 -0500)
committerRobert Haas <rhaas@postgresql.org>
Thu, 5 Jan 2017 19:33:29 +0000 (14:33 -0500)
commit603299167f4aaa857c0156b492ad51143e33fc2f
tree54476301d9608802e211a14739d664cf4838c1a2
parent5b4f8f4c6d618063f7dde0a710a4076fb444620a
Fix possible leak of semaphore count.

Commit 4aec49899e5782247e134f94ce1c6ee926f88e1c reorganized the order
of operations here so that we no longer increment the number of "extra
waits" before locking the semaphore, but it did not change the
starting value of extraWaits from 0 to -1 to compensate.  In the worst
case, this could leak a semaphore count, but that seems to be unlikely
in practice.

Discussion: http://postgr.es/m/CAA4eK1JyVqXiMba+-a589Rk0pyHsyKkGxeumVKjU6Y74hdrVLQ@mail.gmail.com

Amit Kapila, per an off-list report by Dilip Kumar.  Reviewed by me.
src/backend/storage/ipc/procarray.c