Skip to content

Commit ff95114

Browse files
theihorAlexei Starovoitov
authored andcommitted
veristat: Fix a memory leak for preset ENUMERATOR
ASAN detected a memory leak in veristat. The cleanup code handling ENUMERATOR value missed freeing strdup-ed svalue. Fix it. Acked-by: Mykyta Yatsenko <[email protected]> Signed-off-by: Ihor Solodrai <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Alexei Starovoitov <[email protected]>
1 parent 3eb4a2e commit ff95114

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

tools/testing/selftests/bpf/veristat.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3378,6 +3378,8 @@ int main(int argc, char **argv)
33783378
}
33793379
}
33803380
free(env.presets[i].atoms);
3381+
if (env.presets[i].value.type == ENUMERATOR)
3382+
free(env.presets[i].value.svalue);
33813383
}
33823384
free(env.presets);
33833385
return -err;

0 commit comments

Comments
 (0)