commit eef61bb81a22f4c61c01e5afc8bd3b8612a71534
author Florin Coras <fcoras@cisco.com> Tue Aug 27 15:13:35 2019 -0700
committer Damjan Marion <dmarion@me.com> Wed Aug 28 15:04:19 2019 +0000
tree fe89bd11bbc9f755b8fbb52baf785b26590066e5
parent 50059cca899117f7d3d2de616ada05166ccd1c97

session: fix unbind coverity warning

Type:fix

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

src/vnet/session/session_api.c

diff --git a/src/vnet/session/session_api.c b/src/vnet/session/session_api.c
index 8f9ce3f..c55aab3 100755
--- a/src/vnet/session/session_api.c
+++ b/src/vnet/session/session_api.c
@@ -968,10 +968,17 @@
       if ((rv = vnet_unlisten (a)))
 	clib_warning ("unlisten returned: %d", rv);
     }
+  else
+    {
+      rv = VNET_API_ERROR_APPLICATION_NOT_ATTACHED;
+    }
 
 done:
   REPLY_MACRO (VL_API_UNBIND_SOCK_REPLY);
 
+  if (!app)
+    return;
+
   app_wrk = application_get_worker (app, a->wrk_map_index);
   if (!app_wrk)
     return;