misc: fix coverity warnings 70/26870/2
authorDave Barach <dave@barachs.net>
Mon, 4 May 2020 18:47:45 +0000 (14:47 -0400)
committerDamjan Marion <dmarion@me.com>
Wed, 6 May 2020 10:24:21 +0000 (10:24 +0000)
Type: fix
Ticket: VPP-1837

Signed-off-by: Dave Barach <dave@barachs.net>
Change-Id: I51660e4b02f449bd2db12a8cfd395c6c343d2dee

src/vlib/linux/pci.c
src/vnet/lisp-cp/one_api.c

index 8af1cda..722fb88 100644 (file)
@@ -1137,7 +1137,7 @@ vlib_pci_map_region_int (vlib_main_t * vm, vlib_pci_dev_handle_t h,
   if (*result == (void *) -1)
     {
       error = clib_error_return_unix (0, "mmap `BAR%u'", bar);
-      if (p->type == LINUX_PCI_DEVICE_TYPE_UIO)
+      if (p->type == LINUX_PCI_DEVICE_TYPE_UIO && (fd != -1))
        close (fd);
       return error;
     }
index 912b9df..f7a972b 100644 (file)
@@ -1119,7 +1119,7 @@ one_adjacency_copy (vl_api_one_adjacency_t * dst, lisp_adjacency_t * adjs)
          a.leid.address.nsh.si = gid_address_nsh_si (&adj->leid);
          break;
        default:
-         ASSERT (0);
+         ALWAYS_ASSERT (0);
        }
       a.reid.type = eid_type;
       a.leid.type = eid_type;