From 4068f70d2b0cf5491d00cad0a73a70fa385490a1 Mon Sep 17 00:00:00 2001 From: Mathias Fredriksson Date: Mon, 12 Jun 2023 21:38:46 +0300 Subject: [PATCH] fix(coderd): avoid deadlock in `(*logFollower).follow` (#7983) --- coderd/provisionerjobs.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/coderd/provisionerjobs.go b/coderd/provisionerjobs.go index f667c2ec65..99fb647385 100644 --- a/coderd/provisionerjobs.go +++ b/coderd/provisionerjobs.go @@ -318,6 +318,10 @@ func (f *logFollower) follow() { return } defer subCancel() + // Move cancel up the stack so it happens before unsubscribing, + // otherwise we can end up in a deadlock due to how the + // in-memory pubsub does mutex locking on send/unsubscribe. + defer cancel() // we were provided `complete` prior to starting this subscription, so // we also need to check whether the job is now complete, in case the