On Mon, Oct 23, 2023 at 11:28 AM Xavier Simonart <[email protected]> wrote: > > Signed-off-by: Xavier Simonart <[email protected]> > --- > ic/ovn-ic.c | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/ic/ovn-ic.c b/ic/ovn-ic.c > index e2023c2ba..eec466ec4 100644 > --- a/ic/ovn-ic.c > +++ b/ic/ovn-ic.c > @@ -1630,13 +1630,18 @@ collect_lr_routes(struct ic_context *ctx, > const struct icnbrec_transit_switch *key; > > struct hmap *routes_ad; > + const struct icnbrec_transit_switch *t_sw; > for (int i = 0; i < ic_lr->n_isb_pbs; i++) { > isb_pb = ic_lr->isb_pbs[i]; > key = icnbrec_transit_switch_index_init_row( > ctx->icnbrec_transit_switch_by_name); > icnbrec_transit_switch_index_set_name(key, isb_pb->transit_switch); > - ts_name = icnbrec_transit_switch_index_find( > - ctx->icnbrec_transit_switch_by_name, key)->name; > + t_sw = icnbrec_transit_switch_index_find( > + ctx->icnbrec_transit_switch_by_name, key); > + if (!t_sw) { > + continue; > + } > + ts_name = t_sw->name; > icnbrec_transit_switch_index_destroy_row(key); > routes_ad = shash_find_data(routes_ad_by_ts, ts_name); > if (!routes_ad) { > -- > 2.31.1 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev >
Looks good to me, thanks. Acked-by: Ales Musil <[email protected]> -- Ales Musil Senior Software Engineer - OVN Core Red Hat EMEA [email protected] _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
