[PATCH lksctp-tools 0/2] sctp_send/v fixes

[PATCH lksctp-tools 0/2] sctp_send/v fixes

[Marcelo Ricardo Leitner]

First one is more important. Identified by CovScan.
2nd patch is just a cosmetic change.

Marcelo Ricardo Leitner (2):
  sctp_send: fix ignored flags parameter
  sctp_sendv: avoid explicit memset for var initialization

 src/lib/sendmsg.c | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

-- 
2.30.2

[PATCH lksctp-tools 1/2] sctp_send: fix ignored flags parameter

[Marcelo Ricardo Leitner]

CovScan noticed that outmsg.msg_flags was being used uninitialized
if sinfo was not provided. Lets fix it by initializing it with the flags
parameter, like sctp_sendv does.

Fixes: 91239acfc91f ("Add sctp_send() API support and testcases")
Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
---
 src/lib/sendmsg.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/src/lib/sendmsg.c b/src/lib/sendmsg.c
index 4c90b0757f4bb1a337de44cfa974a356fef031e6..c2bd51029964f3bf05a512ea61d479c086f6088c 100644
--- a/src/lib/sendmsg.c
+++ b/src/lib/sendmsg.c
@@ -77,25 +77,21 @@ int
 sctp_send(int s, const void *msg, size_t len,
           const struct sctp_sndrcvinfo *sinfo, int flags)
 {
-	struct msghdr outmsg;
+	struct msghdr outmsg = {};
 	struct iovec iov;
 	char outcmsg[CMSG_SPACE(sizeof(struct sctp_sndrcvinfo))];
 
-	outmsg.msg_name = NULL;
-	outmsg.msg_namelen = 0;
 	outmsg.msg_iov = &iov;
 	iov.iov_base = (void *)msg;
 	iov.iov_len = len;
 	outmsg.msg_iovlen = 1;
-	outmsg.msg_control = NULL;
-	outmsg.msg_controllen = 0;
+	outmsg.msg_flags = flags;
 
 	if (sinfo) {	
 		struct cmsghdr *cmsg;
 
 		outmsg.msg_control = outcmsg;
 		outmsg.msg_controllen = sizeof(outcmsg);
-		outmsg.msg_flags = 0;
 
 		cmsg = CMSG_FIRSTHDR(&outmsg);
 		cmsg->cmsg_level = IPPROTO_SCTP;
-- 
2.30.2

[PATCH lksctp-tools 2/2] sctp_sendv: avoid explicit memset for var initialization

[Marcelo Ricardo Leitner]

Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
---
 src/lib/sendmsg.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/lib/sendmsg.c b/src/lib/sendmsg.c
index c2bd51029964f3bf05a512ea61d479c086f6088c..bee492134e30cbc41ecfce7117444d17c8ab9089 100644
--- a/src/lib/sendmsg.c
+++ b/src/lib/sendmsg.c
@@ -125,14 +125,12 @@ int sctp_sendv(int s, const struct iovec *iov, int iovcnt,
 {
 	char _cmsg[CMSG_SPACE(sizeof(struct sctp_sendv_spa))];
 	struct cmsghdr *cmsg = (struct cmsghdr *)_cmsg;
+	struct msghdr outmsg = {};
 	struct sockaddr *addr;
-	struct msghdr outmsg;
 	int len, cmsglen = 0;
 	int err, type, i;
 	char *addrbuf;
 
-	/* set msg_iov, msg_iovlen, msg_flags */
-	memset(&outmsg, 0x00, sizeof(outmsg));
 	outmsg.msg_iov = (struct iovec *)iov;
 	outmsg.msg_iovlen = iovcnt;
 	outmsg.msg_flags = flags;
-- 
2.30.2

Re: [PATCH lksctp-tools 0/2] sctp_send/v fixes

[Xin Long]

On Thu, Apr 22, 2021 at 3:36 PM Marcelo Ricardo Leitner
<marcelo.leitner@gmail.com> wrote:
>
> First one is more important. Identified by CovScan.
> 2nd patch is just a cosmetic change.
>
> Marcelo Ricardo Leitner (2):
>   sctp_send: fix ignored flags parameter
>   sctp_sendv: avoid explicit memset for var initialization
>
>  src/lib/sendmsg.c | 12 +++---------
>  1 file changed, 3 insertions(+), 9 deletions(-)
>
> --
> 2.30.2
>
Series Reviewed-by: Xin Long <lucien.xin@gmail.com>