On Thu, Sep 01, 2016 at 02:27:55PM -0400, Flavio Fernandes wrote: > > > On Sep 1, 2016, at 1:02 PM, Ben Pfaff <[email protected]> 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 <[email protected]> > > Signed-off-by: Ben Pfaff <[email protected]> > > Acked-by: Flavio Fernandes <[email protected]>
Thanks Flavio and Ryan, I applied this to master and branch-2.6. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
