'timewarp_seq' should be initialized only once, while init_clock() is called
multiple times.

Found by valgrind

Signed-off-by: Daniele Di Proietto <ddiproie...@vmware.com>
---
 lib/timeval.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/timeval.c b/lib/timeval.c
index 0587834..99f9b94 100644
--- a/lib/timeval.c
+++ b/lib/timeval.c
@@ -114,7 +114,6 @@ init_clock(struct clock *c, clockid_t id)
     ovs_mutex_init(&c->mutex);
     atomic_init(&c->slow_path, false);
     xclock_gettime(c->id, &c->cache);
-    timewarp_seq = seq_create();
     memset(&c->large_warp, 0, sizeof(c->large_warp));
 }
 
@@ -125,6 +124,7 @@ do_init_time(void)
 
     coverage_init();
 
+    timewarp_seq = seq_create();
     init_clock(&monotonic_clock, (!clock_gettime(CLOCK_MONOTONIC, &ts)
                                   ? CLOCK_MONOTONIC
                                   : CLOCK_REALTIME));
-- 
2.1.0.rc1

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to