On Thu, Sep 01, 2016 at 02:27:55PM -0400, Flavio Fernandes wrote: > > > On Sep 1, 2016, at 1:02 PM, Ben Pfaff <b...@ovn.org> wrote: > > > > Nothing freed 'reply'. This fixes the problem. > > > > Most of this patch is moving coding around. The essential change is that > > breaking the code that works with 'reply' out into a separate function > > makes it possible to catch all paths out of the function so that 'reply' > > can be freed in one place. > > > > +1 for moving it into a function and calling > "ofputil_uninit_tlv_table(&reply.mappings);" > in a single place! > > > > Reported-by: Ryan Moats <rmo...@us.ibm.com> > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > Acked-by: Flavio Fernandes <fla...@flaviof.com>
Thanks Flavio and Ryan, I applied this to master and branch-2.6. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev