> On Jul 5, 2017, at 4:09 PM, Ben Pfaff <[email protected]> wrote: > > On Tue, Jun 27, 2017 at 06:14:43PM -0700, Justin Pettit wrote: >> The function parse_ofp_meter_mod_str() allocates a buffer called >> 'bands', which parse_ofp_meter_mod_str__() then steals for the member >> 'mm->meter.bands'. Calling functions didn't free that stolen value and >> the comments for those function didn't indicate that was necessary. >> >> Found by valgrind. >> >> Signed-off-by: Justin Pettit <[email protected]> >> --- >> v1->v2: Fix unitialized path. > > I think there's still an uninitialized variable, fold this in?
Good catch. I made that change and pushed it to master. Thanks, --Justin _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
