From ad376c93032c5960392efc3192e2a0852bd16462 Mon Sep 17 00:00:00 2001 From: Joe Stringer Date: Fri, 9 Sep 2016 13:48:52 -0700 Subject: ovsdb-monitor: Fix valgrind 'possible loss' warnings. By placing these nodes at the start of their respective structures, several "possibly lost" warnings from valgrind are silenced. Fixes: 60e0cd041958 ("ovsdb: Replication usability improvements") Signed-off-by: Joe Stringer Acked-by: Ben Pfaff --- ovsdb/monitor.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'ovsdb/monitor.c') diff --git a/ovsdb/monitor.c b/ovsdb/monitor.c index 9a6fbf5a5..5e3c84747 100644 --- a/ovsdb/monitor.c +++ b/ovsdb/monitor.c @@ -86,8 +86,8 @@ struct ovsdb_monitor_json_cache_node { }; struct jsonrpc_monitor_node { - struct ovsdb_jsonrpc_monitor *jsonrpc_monitor; struct ovs_list node; + struct ovsdb_jsonrpc_monitor *jsonrpc_monitor; }; /* A particular column being monitored. */ @@ -116,12 +116,12 @@ struct ovsdb_monitor_row { * 'transaction' stores the first update's transaction id. * */ struct ovsdb_monitor_changes { + struct hmap_node hmap_node; /* Element in ovsdb_monitor_tables' changes + hmap. */ struct ovsdb_monitor_table *mt; struct hmap rows; int n_refs; uint64_t transaction; - struct hmap_node hmap_node; /* Element in ovsdb_monitor_tables' changes - hmap. */ }; /* A particular table being monitored. */ -- cgit v1.2.1