From: Sven Kube Date: Tue, 13 May 2025 14:01:32 +0000 (+0200) Subject: res_audiosocket.c: Add retry mechanism for reading data from AudioSocket X-Git-Tag: 21.10.0-rc1~14 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=76d20593938aa47dc47fd06f822db25d130288c3;p=thirdparty%2Fasterisk.git res_audiosocket.c: Add retry mechanism for reading data from AudioSocket The added retry mechanism addresses an issue that arises when fragmented TCP packets are received, each containing only a portion of an AudioSocket packet. This situation can occur if the external service sending the AudioSocket data has Nagle's algorithm enabled. (cherry picked from commit 0f414f6a9444010daf7986bfd9739a63e6e2be96) --- diff --git a/res/res_audiosocket.c b/res/res_audiosocket.c index 3df074ceaa..daf3e5903d 100644 --- a/res/res_audiosocket.c +++ b/res/res_audiosocket.c @@ -315,6 +315,18 @@ struct ast_frame *ast_audiosocket_receive_frame_with_hangup(const int svc, while (i < *length) { n = read(svc, data + i, *length - i); if (n == -1) { + if (errno == EAGAIN || errno == EWOULDBLOCK) { + int poll_result = ast_wait_for_input(svc, 5); + + if (poll_result == 1) { + continue; + } else if (poll_result == 0) { + ast_log(LOG_WARNING, "Poll timed out while waiting for data\n"); + } else { + ast_log(LOG_WARNING, "Poll error: %s\n", strerror(errno)); + } + } + ast_log(LOG_ERROR, "Failed to read payload from AudioSocket: %s\n", strerror(errno)); ret = -1; break;