sharedpv: avoid deadlock on error path

Need to unlock for all MonitorSetupOp methods
This commit is contained in:
Michael Davidsaver
2022-12-27 08:40:33 -08:00
parent db6b7ba34b
commit b17f820767
+1
View File
@@ -92,6 +92,7 @@ struct SharedPV::Impl : public std::enable_shared_from_this<Impl>
self->subscribers.emplace(std::move(sub));
}catch(std::exception& e){
UnGuard U(G);
log_warn_printf(logshared, "%s Client %s: Can't attach() monitor: %s\n",
conn->name().c_str(), conn->peerName().c_str(), e.what());
// not re-throwing for consistency