]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUILD: mworker: fix ignoring return value of ‘read’ 20250826-unused_result
authorWilliam Lallemand <wlallemand@haproxy.com>
Thu, 28 Aug 2025 13:00:19 +0000 (15:00 +0200)
committerWilliam Lallemand <wlallemand@haproxy.com>
Thu, 28 Aug 2025 13:10:00 +0000 (15:10 +0200)
Fix read return value unused result.

src/haproxy.c: In function ‘main’:
src/haproxy.c:3630:17: error: ignoring return value of ‘read’ declared with attribute ‘warn_unused_result’ [-Werror=unused-result]
 3630 |                 read(sock_pair[1], &c, 1);
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~

Must be backported where d7f6819 is backported.

src/haproxy.c

index b2dec2e3b1cea5d2297b9772106347844a190b94..56789020ed8199dcedea049a756b9f1175b490e5 100644 (file)
@@ -3591,6 +3591,7 @@ int main(int argc, char **argv)
                int sock_pair[2];
                char *msg = NULL;
                char c;
+               int r;
 
                if (socketpair(PF_UNIX, SOCK_STREAM, 0, sock_pair) == -1) {
                        ha_alert("[%s.main()] Cannot create socketpair to update the new worker state\n",
@@ -3626,8 +3627,8 @@ int main(int argc, char **argv)
                 * after confirming receipt of the "\n" from the CLI applet, so
                 * we make sure that the fd is received correctly.
                 */
-                shutdown(sock_pair[1], SHUT_WR);
-               read(sock_pair[1], &c, 1);
+               shutdown(sock_pair[1], SHUT_WR);
+               r = read(sock_pair[1], &c, 1);
                close(sock_pair[1]);
                close(sock_pair[0]);
                ha_free(&msg);