summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2021-02-15 11:56:46 -0800
committerDave Barach <openvpp@barachs.net>2021-02-15 23:42:50 +0000
commitf9db7f0ff51e3c212f70d38c5e4fa68e07b82a96 (patch)
treeb22470ebbbe73835fb7de37d4f852b59918c771e
parenteef15396f69bdb817e9ac2c9b791750be0ce0d4f (diff)
svm: fix mq coverity warning
Type: fix Signed-off-by: Florin Coras <fcoras@cisco.com> Change-Id: I1914366c17fa05305d57d842793fad372319256a
-rw-r--r--MAINTAINERS2
-rw-r--r--src/svm/message_queue.c9
2 files changed, 8 insertions, 3 deletions
diff --git a/MAINTAINERS b/MAINTAINERS
index ea5a28d0aa3..6003a817e30 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -532,7 +532,7 @@ F: src/plugins/mdata/
Plugin - Unit Tests
I: unittest
M: Dave Barach <vpp@barachs.net>
-M: Florin Coras <fcoras@cisco.com
+M: Florin Coras <fcoras@cisco.com>
F: src/plugins/unittest/
Test Infrastructure
diff --git a/src/svm/message_queue.c b/src/svm/message_queue.c
index b423826cb83..8be6be7686f 100644
--- a/src/svm/message_queue.c
+++ b/src/svm/message_queue.c
@@ -570,8 +570,13 @@ svm_msg_q_timedwait (svm_msg_q_t *mq, double timeout)
tv.tv_sec = (u64) timeout;
tv.tv_usec = ((u64) timeout - (u64) timeout) * 1e9;
- setsockopt (mq->q.evtfd, SOL_SOCKET, SO_RCVTIMEO, (const char *) &tv,
- sizeof tv);
+ rv = setsockopt (mq->q.evtfd, SOL_SOCKET, SO_RCVTIMEO,
+ (const char *) &tv, sizeof tv);
+ if (rv < 0)
+ {
+ clib_unix_warning ("setsockopt");
+ return -1;
+ }
rv = read (mq->q.evtfd, &buf, sizeof (buf));
if (rv < 0)