[ovs-git] [openvswitch/ovs] 75aba7: ovsdb-idl: Fix memory leak in ovsdb_idl_condition_...

GitHub noreply at github.com
Tue Aug 16 00:13:18 UTC 2016


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 75aba74df3d4039cb62866e0a46434d3aad72778
      https://github.com/openvswitch/ovs/commit/75aba74df3d4039cb62866e0a46434d3aad72778
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2016-08-15 (Mon, 15 Aug 2016)

  Changed paths:
    M lib/ovsdb-idl.c

  Log Message:
  -----------
  ovsdb-idl: Fix memory leak in ovsdb_idl_condition_add_clause().

The function always allocated a clause but didn't use it if it was
going to be a duplicate.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Flavio Fernandes <flavio at flaviof.com>


  Commit: 239fa5bbe6e54abcec9b58b137a566d06edaba49
      https://github.com/openvswitch/ovs/commit/239fa5bbe6e54abcec9b58b137a566d06edaba49
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2016-08-15 (Mon, 15 Aug 2016)

  Changed paths:
    M lib/ovsdb-idl-provider.h
    M lib/ovsdb-idl.c
    M lib/ovsdb-idl.h

  Log Message:
  -----------
  ovsdb-idl: Style and comment improvements for conditional replication.

The conditional replication code had hardly any comments.  This adds some.

This commit also fixes a number of style problems, factors out some code
into a helper function, and moves some struct declarations from a public
header, that were not used by client code, into more private locations.

Signed-off-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/5f3a48453240...239fa5bbe6e5


More information about the git mailing list