ovsdb-idl: Fix memory leak writing synthetic rows in ovsdb_idl_txn_write().
authorBen Pfaff <blp@nicira.com>
Wed, 28 Mar 2012 21:02:57 +0000 (14:02 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 28 Mar 2012 23:17:05 +0000 (16:17 -0700)
This could cause a slow but steady memory leak in ovs-vswitchd.

Found by valgrind.

Signed-off-by: Ben Pfaff <blp@nicira.com>
lib/ovsdb-idl.c

index 19ae16f..5ad3f5c 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (c) 2009, 2010, 2011 Nicira Networks.
+/* Copyright (c) 2009, 2010, 2011, 2012 Nicira Networks.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -1718,6 +1718,7 @@ ovsdb_idl_txn_write(const struct ovsdb_idl_row *row_,
     size_t column_idx;
 
     if (ovsdb_idl_row_is_synthetic(row)) {
+        ovsdb_datum_destroy(datum, &column->type);
         return;
     }