Fix IC unsubscription race bug (#6304)

alexvru · web-flow · commit d5253416eb92 · 2024-07-04T21:01:52.000Z
diff --git a/ydb/core/blobstorage/nodewarden/distconf_binding.cpp b/ydb/core/blobstorage/nodewarden/distconf_binding.cpp
@@ -144,7 +144,9 @@ namespace NKikimr::NStorage {
         }
 
         const auto it = SubscribedSessions.find(nodeId);
-        Y_ABORT_UNLESS(it != SubscribedSessions.end());
+        if (it == SubscribedSessions.end()) {
+            return; // this may be a race with unsubscription
+        }
         Y_ABORT_UNLESS(!it->second || it->second == ev->Sender);
         SubscribedSessions.erase(it);
 

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,9 @@ namespace NKikimr::NStorage {`
`144`	`144`	`}`
`145`	`145`
`146`	`146`	`const auto it = SubscribedSessions.find(nodeId);`
`147`		`- Y_ABORT_UNLESS(it != SubscribedSessions.end());`
	`147`	`+ if (it == SubscribedSessions.end()) {`
	`148`	`+ return; // this may be a race with unsubscription`
	`149`	`+ }`
`148`	`150`	`Y_ABORT_UNLESS(!it->second \|\| it->second == ev->Sender);`
`149`	`151`	`SubscribedSessions.erase(it);`
`150`	`152`