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