On 7/11/23 12:22, Eelco Chaudron wrote:
> 
> 
> On 13 Jun 2023, at 20:34, James Raphael Tiovalen wrote:
> 
>> In the case where routing is enabled, the bridge member of the
>> `vsctl_port` structs is not populated. This can cause a crash if we
>> attempt to access it. This patch fixes the crash by checking if the
>> bridge member is valid before attempting to access it. In the
>> `check_conflicts` function, we print both the port name and the bridge
>> name if routing is disabled and we only print the port name if routing
>> is enabled.
>>
>> Signed-off-by: James Raphael Tiovalen <jamestio...@gmail.com>
>> Reviewed-by: Simon Horman <simon.hor...@corigine.com>
> 
> Thank for fixing this James, this patch looks good to me.
> 
> Acked-by: Eelco Chaudron <echau...@redhat.com>

Thanks, James, Simon and Eelco!

Applied this one.

Best regards, Ilya Maximets.

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

Reply via email to