K 10
svn:author
V 2
kp
K 8
svn:date
V 27
2023-03-27T08:17:27.812937Z
K 7
svn:log
V 694
pfsync: add missing unlock in pfsync_defer_tmo()

The callout for pfsync_defer_tmo() is created with
CALLOUT_RETURNUNLOCKED, because while the callout framework takes care
of taking the lock we want to run a few operations outside of the lock,
so we unlock ourselves.

However, if `sc->sc_sync_if == NULL` we return without releasing the
lock, and leak the lock, causing later deadlocks.
Ensure we always release the bucket lock when we exit pfsync_defer_tmo()

PR:		268246
MFC after:	1 week
Sponsored by:	Rubicon Communications, LLC (Netgate)

(cherry picked from commit 844ad2828a35c434b893af4274b1f6c50332dd70)

Git Hash:   1d6ece4351a01179e3a5ebbdb3fa2e6053a2d7aa
Git Author: kp@FreeBSD.org
END