Looks good,

Ethan

On Fri, Feb 3, 2012 at 12:38, Ben Pfaff <[email protected]> wrote:
> Found by valgrind.
>
> Signed-off-by: Ben Pfaff <[email protected]>
> ---
>  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
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to