aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorFlorin Coras <fcoras@cisco.com>2021-02-23 12:03:03 -0800
committerFlorin Coras <florin.coras@gmail.com>2021-02-23 19:57:46 +0000
commitd77325c0c43f270a1717d4d5b5acb94cdf674c1f (patch)
treef28a8272f3ebcbbfbfcac01579c1ff48647288a0 /src
parentc2a5ea0b8b41547d9567d35b93407c36b74c7465 (diff)
vcl: fix coverity warning
Type: fix Signed-off-by: Florin Coras <fcoras@cisco.com> Change-Id: Id9f922eafe4a68661d2858d72fc548a372e9596a
Diffstat (limited to 'src')
-rw-r--r--src/vcl/vppcom.c16
1 files changed, 9 insertions, 7 deletions
diff --git a/src/vcl/vppcom.c b/src/vcl/vppcom.c
index a439d2550a1..cc4464b162e 100644
--- a/src/vcl/vppcom.c
+++ b/src/vcl/vppcom.c
@@ -3637,16 +3637,18 @@ vppcom_session_attr (uint32_t session_handle, uint32_t op,
break;
case VPPCOM_ATTR_GET_DOMAIN:
- if (buffer && buflen && (*buflen >= sizeof (int)))
+ if (!(buffer && buflen && (*buflen >= sizeof (int))))
{
- if (session->transport.is_ip4)
- *(int *) buffer = AF_INET;
- else
- *(int *) buffer = AF_INET6;
- *buflen = sizeof (int);
+ rv = VPPCOM_EINVAL;
+ break;
}
+
+ if (session->transport.is_ip4)
+ *(int *) buffer = AF_INET;
else
- rv = VPPCOM_EINVAL;
+ *(int *) buffer = AF_INET6;
+ *buflen = sizeof (int);
+
VDBG (2, "VPPCOM_ATTR_GET_DOMAIN: %d, buflen %u", *(int *) buffer,
*buflen);
break;