diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2024-07-29 18:17:33 +0300 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2024-07-29 18:17:33 +0300 |
| commit | 0393f542d72c6182271c392d9a83d0fc775113c7 (patch) | |
| tree | 29080567907fc09c8371a0b29b35ffc898ed9428 /src/tutorial/funcs.c | |
| parent | 8bda213ec1628737b500b6b04ec164aec354eb04 (diff) | |
Fix double-release of spinlock
Commit 9d9b9d46f3 added spinlocks to protect the fields in ProcSignal
flags, but in EmitProcSignalBarrier(), the spinlock was released
twice. With most spinlock implementations, releasing a lock that's not
held is not easy to notice, because most of the time it does nothing,
but if the spinlock was concurrently acquired by another process, it
could lead to more serious issues. Fortunately, with the
--disable-spinlocks emulation implementation, it caused more visible
failures.
In the passing, fix a type in comment and add an assertion that the
procNumber passed to SendProcSignal looks valid.
Discussion: https://www.postgresql.org/message-id/b8ce284c-18a2-4a79-afd3-1991a2e7d246@iki.fi
Diffstat (limited to 'src/tutorial/funcs.c')
0 files changed, 0 insertions, 0 deletions