Coverity complains that we could potentially dereference a
NULL pointer. To prevent this warning, add an ovs_assert().

Signed-off-by: Eelco Chaudron <echau...@redhat.com>
---
 ofproto/ofproto-dpif-ipfix.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/ofproto/ofproto-dpif-ipfix.c b/ofproto/ofproto-dpif-ipfix.c
index 15b656233..7f10b87a7 100644
--- a/ofproto/ofproto-dpif-ipfix.c
+++ b/ofproto/ofproto-dpif-ipfix.c
@@ -1165,6 +1165,8 @@ dpif_ipfix_set_options(
     struct ofproto_ipfix_flow_exporter_options *options;
     struct dpif_ipfix_flow_exporter_map_node *node;
 
+    ovs_assert(flow_exporters_options || n_flow_exporters_options == 0);
+
     ovs_mutex_lock(&mutex);
     dpif_ipfix_bridge_exporter_set_options(&di->bridge_exporter,
                                            bridge_exporter_options,
-- 
2.47.1

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to