Looks good.

thanx!
mehak

On Fri, Aug 17, 2012 at 2:00 PM, Ben Pfaff <b...@nicira.com> wrote:

> Found by valgrind.
>
> Signed-off-by: Ben Pfaff <b...@nicira.com>
> ---
>  utilities/ovs-ofctl.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/utilities/ovs-ofctl.c b/utilities/ovs-ofctl.c
> index dd5f4ba..622e32f 100644
> --- a/utilities/ovs-ofctl.c
> +++ b/utilities/ovs-ofctl.c
> @@ -2615,6 +2615,7 @@ ofctl_check_vlan(int argc OVS_UNUSED, char *argv[])
>      printf("%04"PRIx16"/%04"PRIx16"\n",
>             ntohs(fm.cr.flow.vlan_tci),
>             ntohs(fm.cr.wc.vlan_tci_mask));
> +    free(string_s);
>
>      /* Convert to and from NXM. */
>      ofpbuf_init(&nxm, 0);
> --
> 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

Reply via email to