Hi,
Please ignore this one [PATCH 2/4], I found it might not be correct and
breaks
testcase 1619: database commands -- wait-until must wait
(make check-valgrind pass, but make check fails)
Regards,
William
On Wed, Jan 6, 2016 at 10:49 AM, William Tu <u9012...@gmail.com> wrote:
> test case 1: appctl-bashcomp - basic verification
> Reason: args used without being free
> Call stacks:
> ds_reserve (dynamic-string.c:63)
> ds_put_uninit (dynamic-string.c:73)
> ds_put_char__ (dynamic-string.c:82)
> ds_put_char (dynamic-string.h:89)
> process_escape_args (process.c:103)
> main (ovs-vsctl.c:150)
>
> Signed-off-by: William Tu <u9012...@gmail.com>
> ---
> utilities/ovs-vsctl.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/utilities/ovs-vsctl.c b/utilities/ovs-vsctl.c
> index 36290db..8c2a252 100644
> --- a/utilities/ovs-vsctl.c
> +++ b/utilities/ovs-vsctl.c
> @@ -2488,6 +2488,7 @@ do_vsctl(const char *args, struct ctl_command
> *commands, size_t n_commands,
> }
>
> ovsdb_idl_txn_add_comment(txn, "ovs-vsctl: %s", args);
> + free(args);
>
> ovs = ovsrec_open_vswitch_first(idl);
> if (!ovs) {
> --
> 2.5.0
>
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
