diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2021-03-18 11:08:16 +1000 |
---|---|---|
committer | Ran Benita <ran@unusedvar.com> | 2021-03-19 23:44:30 +0200 |
commit | 7a205e25ac3b3a69bb12316a096eb1bab35a2702 (patch) | |
tree | f862b12dc3c8acbd8d3ae31022e35730a3a737ec | |
parent | 21c864ccbf9368fe0c504c017bf8f8f414bc2f89 (diff) | |
download | xorg-lib-libxkbcommon-7a205e25ac3b3a69bb12316a096eb1bab35a2702.tar.gz |
bench: silence coverity complaint about a double free
False positive because we rely on xkb_components_from_rules() to initalize this
struct, but let's localize the variable anyway to shut coverity up.
libxkbcommon-1.0.3/bench/rules.c:59:9: warning[-Wanalyzer-double-free]:
double-free of kccgst.symbols
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
-rw-r--r-- | bench/rules.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/bench/rules.c b/bench/rules.c index 3206583..1306dc4 100644 --- a/bench/rules.c +++ b/bench/rules.c @@ -40,7 +40,6 @@ main(int argc, char *argv[]) struct xkb_rule_names rmlvo = { "evdev", "pc105", "us,il", ",", "ctrl:nocaps,grp:menu_toggle", }; - struct xkb_component_names kccgst; struct bench bench; char *elapsed; @@ -52,6 +51,8 @@ main(int argc, char *argv[]) bench_start(&bench); for (i = 0; i < BENCHMARK_ITERATIONS; i++) { + struct xkb_component_names kccgst; + assert(xkb_components_from_rules(ctx, &rmlvo, &kccgst)); free(kccgst.keycodes); free(kccgst.types); |