[ovs-dev] [PATCH] ofctrl: Fix logic for mff_ovn_geneve.

[Guru Shetty]

On 9 September 2016 at 15:49, Ben Pfaff <blp at ovn.org> wrote:

> On Fri, Sep 09, 2016 at 05:38:16AM -0700, Gurucharan Shetty wrote:
> > Fixes: 7fa39c6b9376e99 ("Fix memory leak in
> recv_S_TLV_TABLE_REQUESTED().")
> > Signed-off-by: Gurucharan Shetty <guru at ovn.org>
> > ---
> >  ovn/controller/ofctrl.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/ovn/controller/ofctrl.c b/ovn/controller/ofctrl.c
> > index fe72d79..6ea593b 100644
> > --- a/ovn/controller/ofctrl.c
> > +++ b/ovn/controller/ofctrl.c
> > @@ -220,7 +220,7 @@ process_tlv_table_reply(const struct
> ofputil_tlv_table_reply *reply)
> >              } else {
> >                  mff_ovn_geneve = MFF_TUN_METADATA0 + map->index;
> >                  state = S_CLEAR_FLOWS;
> > -                return false;
> > +                return true;
> >              }
> >          }
>
> Good catch, thanks.
>
> Acked-by: Ben Pfaff <blp at ovn.org>
>

Thank you for the reviews. I applied this to master and 2.6