bus1 / dbus-broker

Linux D-Bus Message Broker
https://github.com/bus1/dbus-broker/wiki
Apache License 2.0
675 stars 78 forks source link

bus/match: flush monitor matches on disconnect #246

Closed dvdhrm closed 3 years ago

dvdhrm commented 3 years ago

(CC: @benzea @teg #245)

We did not correctly flush monitor matches on peer disconnect, but only the normal subscription objects. Fix this and properly flush all matches in a registry on disconnect.

See the commit message for background information.

I have not tested this, yet. But I think this is quite likely the bug @benzea saw. Will report back when I tested it.

dvdhrm commented 3 years ago

Indeed, I verified it now. This does fix the issue and with this applied I cannot reproduce it anymore. I will also merge the audit-3.0 changes once they released their new version and then prep a dbus-broker release next week.