On Wed, Dec 23, 2015 at 10:58:15AM -0800, William Tu wrote:
> perf_counter_accumulate() is invoked without perf_counters_init() being
> called first, which leads to a memory leak reported by Valgrind (test
> cases 104, 106, and 107). A call trace is below:
>     xmalloc (util.c:112)
>     shash_add_nocopy__ (shash.c:109)
>     shash_add_nocopy (shash.c:121)
>     shash_add (shash.c:129)
>     shash_add_once (shash.c:136)
>     shash_add_assert (shash.c:146)
>     perf_counter_init (perf-counter.c:86)
>     perf_counter_accumulate (perf-counter.c:95)
>     ovsdb_txn_commit (transaction.c:850)
>     ovsdb_file_open__ (file.c:217)
>     open_db (ovsdb-server.c:418)
>     main (ovsdb-server.c:263)
> 
> Signed-off-by: William Tu <[email protected]>
> Signed-off-by: Daniele Di Proietto <[email protected]>
> Co-authored-by: Daniele Di Proietto <[email protected]>

Thanks, applied to master and branch-2.5.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to