Drop slot's LWLock before returning from SaveSlotToPath()

Peter Eisentraut <peter@eisentraut.org>

Commit: 2d8597ba66a1646e353c0ff755b4b0b85cd1747f
Author: Peter Eisentraut <peter@eisentraut.org>
Date: 2020-03-26T13:01:56Z
Releases: 9.6.18
Drop slot's LWLock before returning from SaveSlotToPath()

When SaveSlotToPath() is called with elevel=LOG, the early exits didn't
release the slot's io_in_progress_lock.

This could result in a walsender being stuck on the lock forever.  A
possible way to get into this situation is if the offending code paths
are triggered in a low disk space situation.

Author: Pavan Deolasee <pavan.deolasee@2ndquadrant.com>
Reported-by: Craig Ringer <craig@2ndquadrant.com>
Discussion: https://www.postgresql.org/message-id/flat/56a138c5-de61-f553-7e8f-6789296de785%402ndquadrant.com

Files

PathChange+/−
src/backend/replication/slot.c modified +9 −0

Discussion