vcl: fix coverity warning 18/31418/2
authorFlorin Coras <fcoras@cisco.com>
Tue, 23 Feb 2021 20:03:03 +0000 (12:03 -0800)
committerFlorin Coras <florin.coras@gmail.com>
Tue, 23 Feb 2021 19:57:46 +0000 (19:57 +0000)
Type: fix

Signed-off-by: Florin Coras <fcoras@cisco.com>
Change-Id: Id9f922eafe4a68661d2858d72fc548a372e9596a

src/vcl/vppcom.c

index a439d25..cc4464b 100644 (file)
@@ -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;