Thank you. I pushed it.
On Fri, Feb 03, 2012 at 12:40:06PM -0800, Ethan Jackson wrote: > Looks good, > > Ethan > > On Fri, Feb 3, 2012 at 12:38, Ben Pfaff <b...@nicira.com> wrote: > > Found by valgrind. > > > > Signed-off-by: Ben Pfaff <b...@nicira.com> > > --- > > utilities/ovs-ofctl.c | 4 ++++ > > 1 files changed, 4 insertions(+), 0 deletions(-) > > > > diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c > > index 9b1a1b2..7df0a90 100644 > > --- a/utilities/ovs-ofctl.c > > +++ b/utilities/ovs-ofctl.c > > @@ -447,6 +447,8 @@ fetch_switch_config(struct vconn *vconn, struct > ofp_switch_config *config_) > > > > config = reply->data; > > *config_ = *config; > > + > > + ofpbuf_delete(reply); > > } > > > > static void > > @@ -869,6 +871,8 @@ monitor_vconn(struct vconn *vconn) > > unixctl_server_wait(server); > > poll_block(); > > } > > + vconn_close(vconn); > > + unixctl_server_destroy(server); > > } > > > > static void > > -- > > 1.7.2.5 > > > > _______________________________________________ > > dev mailing list > > dev@openvswitch.org > > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
