diff options
author | Joe Stringer <joe@ovn.org> | 2016-09-09 13:48:53 -0700 |
---|---|---|
committer | Joe Stringer <joe@ovn.org> | 2016-09-12 11:05:01 -0700 |
commit | c95e1d65160a9adece38e96515fc5a67a01464e2 (patch) | |
tree | 52dfa7bdd83fe51bdf73813f74b708a31e8de375 /ovsdb | |
parent | ad376c93032c5960392efc3192e2a0852bd16462 (diff) | |
download | openvswitch-c95e1d65160a9adece38e96515fc5a67a01464e2.tar.gz |
ovsdb: Fix replication memory leak.
Valgrind reports:
==18725== 32 bytes in 1 blocks are definitely lost in loss record 339 of 497
==18725== at 0x4C29BBE: malloc (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==18725== by 0x450F1F: xmalloc (util.c:112)
==18725== by 0x41748E: replication_add_local_db (replication.c:137)
==18725== by 0x40803B: ovsdb_replication_init (ovsdb-server.c:146)
==18725== by 0x407C9E: ovsdb_server_connect_active_ovsdb_server
(ovsdb-server.c:1165)
==18725== by 0x450AB3: process_command (unixctl.c:313)
==18725== by 0x4500DC: run_connection (unixctl.c:347)
==18725== by 0x44FFB6: unixctl_server_run (unixctl.c:400)
==18725== by 0x4081AC: main_loop (ovsdb-server.c:182)
==18725== by 0x406432: main (ovsdb-server.c:429)
Fixes: 60e0cd041958 ("ovsdb: Replication usability improvements")
Signed-off-by: Joe Stringer <joe@ovn.org>
Acked-by: Flavio Fernandes <flavio@flaviof.com>
Acked-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'ovsdb')
-rw-r--r-- | ovsdb/replication.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ovsdb/replication.c b/ovsdb/replication.c index c2f9dfb3a..50abf001f 100644 --- a/ovsdb/replication.c +++ b/ovsdb/replication.c @@ -134,7 +134,6 @@ replication_init(const char *sync_from_, const char *exclude_tables) void replication_add_local_db(const char *database, struct ovsdb *db) { - struct shash_node *node = xmalloc(sizeof *node); shash_add_assert(&local_dbs, database, db); } |