On Sat, Dec 5, 2020 at 1:10 AM Dumitru Ceara <dce...@redhat.com> wrote:
>
> On 12/4/20 7:50 PM, Ilya Maximets wrote:
> > During refactoring, direct access such as 'lb->vips[i].backends[j].op'
> > was simplified by the presence of the 'backend_nb' pointer. But instead
> > of the inner one, the index of the outer loop was used.  This forces
> > northd to only use one backend per vip, and can also lead to invalid
> > memory accesses.
> >
> > This change fixes system test:
> >   24. ovn -- Load balancer health checks
> >
> > Fixes: f1119c125765 ("northd: Refactor load balancer vip parsing.")
> > Tested-by: Lorenzo Bianconi <lorenzo.bianc...@redhat.com>
> > Signed-off-by: Ilya Maximets <i.maxim...@ovn.org>
> > ---
>
> Thanks for the fix!
>
> Acked-by: Dumitru Ceara <dce...@redhat.com>

Thanks Ilya for the fix and Dumitru for the review. I applied this
patch to master and branch-20.12.

Numan

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

Reply via email to