On Mon, Aug 15, 2016 at 03:08:04PM -0400, Flaviof wrote:
> On Sat, Aug 13, 2016 at 7:58 PM, Ben Pfaff wrote:
>
> > The function always allocated a clause but didn't use it if it was
> > going to be a duplicate.
> >
> > Signed-off-by: Ben Pfaff
> >
>
> Acked-by: Flavio Fernandes
Thanks, applie
On Sat, Aug 13, 2016 at 7:58 PM, Ben Pfaff wrote:
> The function always allocated a clause but didn't use it if it was
> going to be a duplicate.
>
> Signed-off-by: Ben Pfaff
>
Acked-by: Flavio Fernandes
> ---
> lib/ovsdb-idl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> di
The function always allocated a clause but didn't use it if it was
going to be a duplicate.
Signed-off-by: Ben Pfaff
---
lib/ovsdb-idl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
index ad0d1e1..7d3d328 100644
--- a/lib/ovsdb-idl.c
+++ b
Thanks. I pushed this to master.
The leak is present on other branches too but it's so minor that I'm not
sure it's worth backporting.
On Wed, Aug 15, 2012 at 05:35:47PM -0700, Mehak Mahajan wrote:
> Hey Ben,
>
> Looks good.
>
> thanx!
> mehak
>
> On Wed, Aug 15, 2012 at 4:19 PM, Ben Pfaff w
Hey Ben,
Looks good.
thanx!
mehak
On Wed, Aug 15, 2012 at 4:19 PM, Ben Pfaff wrote:
> Found by valgrind.
>
> Signed-off-by: Ben Pfaff
> ---
> lib/ovsdb-idl.c |1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
> index 534beb0..61188
Found by valgrind.
Signed-off-by: Ben Pfaff
---
lib/ovsdb-idl.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
index 534beb0..6118852 100644
--- a/lib/ovsdb-idl.c
+++ b/lib/ovsdb-idl.c
@@ -230,6 +230,7 @@ ovsdb_idl_destroy(struct ovsdb_
