xartigas pushed a commit to branch master. http://git.enlightenment.org/core/efl.git/commit/?id=db195e76cdca0f7602b675eeb814facad1afd800
commit db195e76cdca0f7602b675eeb814facad1afd800 Author: Christopher Michael <[email protected]> Date: Thu May 2 13:46:33 2019 +0200 eina_bench_stringshare: Fix resourcce leak Summary: Coverity reports that we leak the return from eina_counter_dump here, so store the result, print it out, then we can free it. Fixes CID1400975 @fix Depends on D8767 Reviewers: raster, cedric, zmike, bu5hm4n, segfaultxavi Reviewed By: segfaultxavi Subscribers: segfaultxavi, #reviewers, #committers Tags: #efl Differential Revision: https://phab.enlightenment.org/D8768 --- src/benchmarks/eina/eina_bench_stringshare_e17.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/benchmarks/eina/eina_bench_stringshare_e17.c b/src/benchmarks/eina/eina_bench_stringshare_e17.c index 2e2769641e..2b2b45cb34 100644 --- a/src/benchmarks/eina/eina_bench_stringshare_e17.c +++ b/src/benchmarks/eina/eina_bench_stringshare_e17.c @@ -81,6 +81,7 @@ static void eina_bench_e17_stringshare(Eina_Stringshare_Test *str) { Eina_Counter *cnt; + char *result; cnt = eina_counter_new(str->name); @@ -96,7 +97,9 @@ eina_bench_e17_stringshare(Eina_Stringshare_Test *str) eina_counter_stop(cnt, 1); - fprintf(stderr, "For `%s`:\n%s\n", str->name,eina_counter_dump(cnt)); + result = eina_counter_dump(cnt); + fprintf(stderr, "For `%s`:\n%s\n", str->name, result); + free(result); eina_counter_free(cnt); } --
