[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 09/10] python/aqmp: fix race condition in legacy.py
From: |
John Snow |
Subject: |
[PATCH 09/10] python/aqmp: fix race condition in legacy.py |
Date: |
Fri, 25 Feb 2022 15:59:47 -0500 |
With all of that refactoring out of the way, this is extraordinarily
simple.
Signed-off-by: John Snow <jsnow@redhat.com>
---
python/qemu/aqmp/legacy.py | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/python/qemu/aqmp/legacy.py b/python/qemu/aqmp/legacy.py
index dca1e76ed4..cb50e60564 100644
--- a/python/qemu/aqmp/legacy.py
+++ b/python/qemu/aqmp/legacy.py
@@ -57,7 +57,7 @@ def __init__(self, address: SocketAddrT,
self._timeout: Optional[float] = None
if server:
- self._aqmp._bind_hack(address) # pylint: disable=protected-access
+ self._sync(self._aqmp.start_server(address))
_T = TypeVar('_T')
@@ -90,10 +90,7 @@ def accept(self, timeout: Optional[float] = 15.0) ->
QMPMessage:
self._aqmp.await_greeting = True
self._aqmp.negotiate = True
- self._sync(
- self._aqmp.start_server_and_accept(self._address),
- timeout
- )
+ self._sync(self._aqmp.accept(), timeout)
ret = self._get_greeting()
assert ret is not None
--
2.34.1
- [PATCH 00/10] Python: Fix qmp race condition on accept(), John Snow, 2022/02/25
- [PATCH 02/10] python/aqmp: rename 'accept()' to 'start_server_and_accept()', John Snow, 2022/02/25
- [PATCH 01/10] python/aqmp: add _session_guard(), John Snow, 2022/02/25
- [PATCH 03/10] python/aqmp: remove _new_session and _establish_connection, John Snow, 2022/02/25
- [PATCH 06/10] python/aqmp: refactor _do_accept() into two distinct steps, John Snow, 2022/02/25
- [PATCH 05/10] python/aqmp: squelch pylint warning for too many lines, John Snow, 2022/02/25
- [PATCH 04/10] python/aqmp: split _client_connected_cb() out as _incoming(), John Snow, 2022/02/25
- [PATCH 09/10] python/aqmp: fix race condition in legacy.py,
John Snow <=
- [PATCH 07/10] python/aqmp: stop the server during disconnect(), John Snow, 2022/02/25
- [PATCH 10/10] python/aqmp: drop _bind_hack(), John Snow, 2022/02/25
- [PATCH 08/10] python/aqmp: add start_server() and accept() methods, John Snow, 2022/02/25