fix: tolerate removing unregistered listener (GHSA-5pv9-xcmm-gqc7)

bluetoothbot · bluetoothbot · commit fbed0b869665 · 2026-05-26T16:55:21.000Z
self.listeners is a set, so .remove() raises KeyError, never ValueError —
the existing except clause could not catch the only realistic failure
mode. Switch to the set-native discard idiom so removing a listener that
was never registered (e.g. during teardown / reconnect sequences) no
longer breaks the shutdown flow.
diff --git a/src/zeroconf/_handlers/record_manager.py b/src/zeroconf/_handlers/record_manager.py
@@ -214,8 +214,6 @@ def async_remove_listener(self, listener: RecordUpdateListener) -> None:
 
         This function is not threadsafe and must be called in the eventloop.
         """
-        try:
-            self.listeners.remove(listener)
+        if listener in self.listeners:
+            self.listeners.discard(listener)
             self.zc.async_notify_all()
-        except ValueError as e:
-            log.exception("Failed to remove listener: %r", e)
