On 2/2/23 01:30, Han Zhou wrote:
> In ovn_igmp_group_get_ports(), it accesses a union member that should
> exist only if the port is a LSP: port->peer->od->mcast_info.rtr.relay.
> But in theory it is possible that the "port" is in fact a LRP, because
> it is a result of ovn_port_find() with the port name coming from a SB DB
> entry. So it is better to validate that first.
> 
> Signed-off-by: Han Zhou <[email protected]>
> ---

Good catch, thanks for the fix!

Acked-by: Dumitru Ceara <[email protected]>

Do you think it's worth backporting this?

Regards,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to