Merge pull request #192 from attestantio/lock-subscriptions-info

Avoid crash with subscriptionsInfo.
attestantio · Mar 18, 2024 · 1fb1852 · 1fb1852
2 parents 03ccf88 + c44450f
commit 1fb1852
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,4 +1,4 @@
-dev:
+1.9.0-alpha.1:
   - reduce time spent verifying account names
   - ensure that attestations complete on Vouch's first ever epoch
   - allow Vouch to start with some consensus nodes unavailable

diff --git a/main.go b/main.go
@@ -108,7 +108,7 @@ import (
 )
 
 // ReleaseVersion is the release version for the code.
-var ReleaseVersion = "1.9.0-dev"
+var ReleaseVersion = "1.9.0-alpha.1"
 
 func main() {
 	exitCode := main2()

diff --git a/services/controller/standard/events.go b/services/controller/standard/events.go
@@ -140,7 +140,9 @@ func (s *Service) HandleHeadEvent(event *api.Event) {
 	}
 
 	// Remove old subscriptions if present.
+	s.subscriptionInfosMutex.Lock()
 	delete(s.subscriptionInfos, s.chainTimeService.SlotToEpoch(data.Slot)-2)
+	s.subscriptionInfosMutex.Unlock()
 }
 
 // handlePreviousDependentRootChanged handles the situation where the previous