diff mbox series

[2/3] qemurunner: avoid leaking server_socket

Message ID 20230510125935.4120156-2-mikko.rapeli@linaro.org
State Accepted, archived
Commit be7bbbeeb44158b181d1abc5a4d7871a9045fdd6
Headers show
Series [1/3] qemurunner: support serial console login via qemu stdout | expand

Commit Message

Mikko Rapeli May 10, 2023, 12:59 p.m. UTC
close() the server_socket before qemusock replaces
it.

Signed-off-by: Mikko Rapeli <mikko.rapeli@linaro.org>
---
 meta/lib/oeqa/utils/qemurunner.py | 1 +
 1 file changed, 1 insertion(+)

Comments

Philippe Mathieu-Daudé May 22, 2023, 10:09 a.m. UTC | #1
On 10/5/23 14:59, Mikko Rapeli wrote:
> close() the server_socket before qemusock replaces
> it.
> 
> Signed-off-by: Mikko Rapeli <mikko.rapeli@linaro.org>
> ---
>   meta/lib/oeqa/utils/qemurunner.py | 1 +
>   1 file changed, 1 insertion(+)

Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
diff mbox series

Patch

diff --git a/meta/lib/oeqa/utils/qemurunner.py b/meta/lib/oeqa/utils/qemurunner.py
index 3b7398b872..c06268f5bc 100644
--- a/meta/lib/oeqa/utils/qemurunner.py
+++ b/meta/lib/oeqa/utils/qemurunner.py
@@ -491,6 +491,7 @@  class QemuRunner:
 
                         decodedlog = self.decode_qemulog(bootlog)
                         if self.boot_patterns['search_reached_prompt'] in decodedlog:
+                            self.server_socket.close()
                             self.server_socket = qemusock
                             stopread = True
                             reachedlogin = True