backends/winrt/client: fix crash in max_pdu_size_changed_handler

It is possible that self._session is set to None when max_pdu_size_changed_handler is called. So we can use the sender arg instead to avoid a possible AttributeError. Fixes #998.
hbldh · Sep 12, 2022 · 21e72cb · 21e72cb
1 parent d81fd3d
commit 21e72cb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -34,6 +34,7 @@ Fixed
 * Fixed use of wrong enum in unpair function of WinRT backend.
 * Fixed inconsistent return types for ``properties`` and ``descriptors`` properties of ``BleakGATTCharacteristic``.
 * Handle device being removed before GetManagedObjects returns in BlueZ backend. Fixes #996.
+* Fixed crash in ``max_pdu_size_changed_handler`` in WinRT backend. Fixes #998.
 
 Removed
 -------

diff --git a/bleak/backends/winrt/client.py b/bleak/backends/winrt/client.py
@@ -279,7 +279,7 @@ def session_status_changed_event_handler(
             loop.call_soon_threadsafe(handle_session_status_changed, args)
 
         def max_pdu_size_changed_handler(sender: GattSession, args):
-            logger.debug("max_pdu_size_changed_handler: %d", self._session.max_pdu_size)
+            logger.debug("max_pdu_size_changed_handler: %d", sender.max_pdu_size)
 
         # Start a GATT Session to connect
         event = asyncio.Event()