diff --git a/ospf6d/ospf6_spf.c b/ospf6d/ospf6_spf.c
index 17ce1771e2..29ba1bcec7 100644
--- a/ospf6d/ospf6_spf.c
+++ b/ospf6d/ospf6_spf.c
@@ -1029,18 +1029,21 @@ struct ospf6_lsa *ospf6_create_single_router_lsa(struct ospf6_area *area,
 
 	/* Fill Larger LSA Payload */
 	end = ospf6_lsdb_head(lsdb, 2, type, adv_router, &rtr_lsa);
-	if (rtr_lsa) {
-		if (!OSPF6_LSA_IS_MAXAGE(rtr_lsa)) {
-			/* Append first Link State ID LSA */
-			lsa_header = (struct ospf6_lsa_header *)rtr_lsa->header;
-			memcpy(new_header, lsa_header,
-				ntohs(lsa_header->length));
-			/* Assign new lsa length as aggregated length. */
-			((struct ospf6_lsa_header *)new_header)->length =
-					htons(total_lsa_length);
-			new_header += ntohs(lsa_header->length);
-			num_lsa--;
-		}
+
+	/*
+	 * We assume at this point in time that rtr_lsa is
+	 * a valid pointer.
+	 */
+	assert(rtr_lsa);
+	if (!OSPF6_LSA_IS_MAXAGE(rtr_lsa)) {
+		/* Append first Link State ID LSA */
+		lsa_header = (struct ospf6_lsa_header *)rtr_lsa->header;
+		memcpy(new_header, lsa_header, ntohs(lsa_header->length));
+		/* Assign new lsa length as aggregated length. */
+		((struct ospf6_lsa_header *)new_header)->length =
+			htons(total_lsa_length);
+		new_header += ntohs(lsa_header->length);
+		num_lsa--;
 	}
 
 	/* Print LSA Name */
diff --git a/ospfd/ospf_ext.c b/ospfd/ospf_ext.c
index d42476b6d8..d7faf4b0de 100644
--- a/ospfd/ospf_ext.c
+++ b/ospfd/ospf_ext.c
@@ -175,7 +175,7 @@ void ospf_ext_term(void)
 {
 
 	if ((OspfEXT.scope != OSPF_OPAQUE_AREA_LSA)
-	    || (OspfEXT.scope != OSPF_OPAQUE_AS_LSA))
+	    && (OspfEXT.scope != OSPF_OPAQUE_AS_LSA))
 		zlog_warn(
 			"EXT: Unable to unregister Extended Prefix "
 			"Opaque LSA functions: Wrong scope!");
diff --git a/ospfd/ospf_flood.c b/ospfd/ospf_flood.c
index 36b6d5143d..7ad9cf9f2f 100644
--- a/ospfd/ospf_flood.c
+++ b/ospfd/ospf_flood.c
@@ -563,6 +563,7 @@ int ospf_flood_through_area(struct ospf_area *area, struct ospf_neighbor *inbr,
 	struct ospf_interface *oi;
 	int lsa_ack_flag = 0;
 
+	assert(area);
 	/* All other types are specific to a single area (Area A).  The
 	   eligible interfaces are all those interfaces attaching to the
 	   Area A.  If Area A is the backbone, this includes all the virtual
diff --git a/ospfd/ospf_sr.c b/ospfd/ospf_sr.c
index c6649a7a04..9827eac71b 100644
--- a/ospfd/ospf_sr.c
+++ b/ospfd/ospf_sr.c
@@ -149,14 +149,6 @@ static struct sr_node *sr_node_new(struct in_addr *rid)
 	new->ext_link->del = del_sr_link;
 	new->ext_prefix->del = del_sr_pref;
 
-	/* Check if list are correctly created */
-	if (new->ext_link == NULL || new->ext_prefix == NULL) {
-		list_delete_original(new->ext_link);
-		list_delete_original(new->ext_prefix);
-		XFREE(MTYPE_OSPF_SR_PARAMS, new);
-		return NULL;
-	}
-
 	IPV4_ADDR_COPY(&new->adv_router, rid);
 	new->neighbor = NULL;
 	new->instance = 0;
@@ -440,7 +432,7 @@ static struct ospf_path *get_nexthop_by_addr(struct ospf *top,
 	struct route_node *rn;
 
 	/* Sanity Check */
-	if ((top == NULL) && (top->new_table))
+	if (top == NULL)
 		return NULL;
 
 	if (IS_DEBUG_OSPF_SR)
diff --git a/zebra/zebra_ptm.c b/zebra/zebra_ptm.c
index 7f5fd472f1..187c2594ad 100644
--- a/zebra/zebra_ptm.c
+++ b/zebra/zebra_ptm.c
@@ -1007,8 +1007,13 @@ int zebra_ptm_bfd_client_register(struct zserv *client,
 	return 0;
 
 stream_failure:
-	if (out_ctxt)
-		ptm_lib_cleanup_msg(ptm_hdl, out_ctxt);
+	/*
+	 * IF we ever add more STREAM_GETXXX functions after the out_ctxt
+	 * is allocated then we need to add this code back in
+	 *
+	 * if (out_ctxt)
+	 *	ptm_lib_cleanup_msg(ptm_hdl, out_ctxt);
+	 */
 	return 0;
 }
 
diff --git a/zebra/zebra_vty.c b/zebra/zebra_vty.c
index f5ad9c1c8a..269244f768 100644
--- a/zebra/zebra_vty.c
+++ b/zebra/zebra_vty.c
@@ -458,6 +458,12 @@ DEFPY(ip_route_blackhole_vrf,
 	VTY_DECLVAR_CONTEXT(vrf, vrf);
 	struct zebra_vrf *zvrf = vrf->info;
 
+	/*
+	 * Coverity is complaining that prefix could
+	 * be dereferenced, but we know that prefix will
+	 * valid.  Add an assert to make it happy
+	 */
+	assert(prefix);
 	return zebra_static_route_leak(vty, zvrf, zvrf,
 				       AFI_IP, SAFI_UNICAST, no, prefix,
 				       mask_str, NULL, NULL, NULL, flag,
@@ -2014,6 +2020,12 @@ DEFPY(ipv6_route_blackhole_vrf,
 	VTY_DECLVAR_CONTEXT(vrf, vrf);
 	struct zebra_vrf *zvrf = vrf->info;
 
+	/*
+	 * Coverity is complaining that prefix could
+	 * be dereferenced, but we know that prefix will
+	 * valid.  Add an assert to make it happy
+	 */
+	assert(prefix);
 	return zebra_static_route_leak(vty, zvrf, zvrf,
 				       AFI_IP6, SAFI_UNICAST, no, prefix_str,
 				       NULL, from_str, NULL, NULL, flag,