]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: quic: initialize msg_flags before sendmsg
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Wed, 21 Feb 2024 09:05:14 +0000 (10:05 +0100)
committerAmaury Denoyelle <adenoyelle@haproxy.com>
Wed, 21 Feb 2024 09:13:53 +0000 (10:13 +0100)
Previously, msghdr struct used for sendmsg was memset to 0. This was
updated for performance reason with each members individually defined.
This is done by the following commit :

  commit 107d6d75465419a09d90c790edb617091a04468a
  OPTIM: quic: improve slightly qc_snd_buf() internal

msg_flags is the only member unset, as sendmsg manual page reports that
it is unused. However, this caused a coverity report. In the end, it is
better to explicitely set it to 0 to avoid any future interrogations,
compiler warning or even portability issues.

This should fix coverity report from github issue #2455.

No need to backport unless above patch is.

src/quic_sock.c

index 6d80b0450d43f88241de565cd06bd6bab99a62d5..4d51ef6fdebf90c06364fb29c94e332395b8e227 100644 (file)
@@ -700,6 +700,7 @@ int qc_snd_buf(struct quic_conn *qc, const struct buffer *buf, size_t sz,
        msg.msg_iovlen = 1;
        msg.msg_control = NULL;
        msg.msg_controllen = 0;
+       msg.msg_flags = 0;
 
        if (qc_test_fd(qc) && !fd_send_ready(qc->fd))
                return 0;