From: Nick Porter <nick@portercomputing.co.uk>
Date: Tue, 27 Jan 2026 08:49:59 +0000 (+0000)
Subject: Pacify Coverity (CID #1665464)
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=933884c90456f48fb50a47e0607560baf6bcf33e;p=thirdparty%2Ffreeradius-server.git

Pacify Coverity (CID #1665464)

The number of bytes read is not important - it is simply a trigger to
pop all available messages from the atomic queue.
---

diff --git a/src/lib/io/control.c b/src/lib/io/control.c
index 86a847ed320..46d1b3e951a 100644
--- a/src/lib/io/control.c
+++ b/src/lib/io/control.c
@@ -97,6 +97,12 @@ static void pipe_read(UNUSED fr_event_list_t *el, int fd, UNUSED int flags, void
 	size_t message_size;
 	uint32_t id = 0;
 
+	/*
+	 *  The presence of data on the pipe fd is just a trigger to pop all
+	 *  available messages from the atomic queue, so the number of bytes
+	 *  read is not important.
+	 */
+	/* coverity[check_return] */
 	if (read(fd, read_buffer, sizeof(read_buffer)) <= 0) return;
 
 	now = fr_time();