From ad6c107f91047ccd8a272516f0f1096fb144090c Mon Sep 17 00:00:00 2001 From: Louis Scalbert Date: Wed, 23 Oct 2024 12:25:42 +0200 Subject: [PATCH] tests: bmpserver, detect session close immediately bmpserver infinitely loops after the clients has closed the TCP session. In this situation, recv() returns empty data. Detect session close immediately. Fixes: 875511c466 ("topotests: add basic bmp collector") Signed-off-by: Louis Scalbert --- tests/topotests/lib/bmp_collector/bmpserver | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/topotests/lib/bmp_collector/bmpserver b/tests/topotests/lib/bmp_collector/bmpserver index 944f299a03..264a281c49 100755 --- a/tests/topotests/lib/bmp_collector/bmpserver +++ b/tests/topotests/lib/bmp_collector/bmpserver @@ -34,6 +34,9 @@ def main(): try: while True: data = connection.recv(BGP_MAX_SIZE) + if not data: + # connection closed + break while len(data) > BMPMsg.MIN_LEN: data = BMPMsg.dissect(data, log_file=LOG_FILE) except Exception as e: