Commit b8b26e58 authored by Andrii Nakryiko's avatar Andrii Nakryiko Committed by Alexei Starovoitov

selftests/bpf: Free per-cpu values array in bpf_iter selftest

Array holding per-cpu values wasn't freed. Fix that.
Signed-off-by: default avatarAndrii Nakryiko <andrii@kernel.org>
Signed-off-by: default avatarAlexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/20211107165521.9240-5-andrii@kernel.org
parent 8ba28587
...@@ -699,14 +699,13 @@ static void test_bpf_percpu_hash_map(void) ...@@ -699,14 +699,13 @@ static void test_bpf_percpu_hash_map(void)
char buf[64]; char buf[64];
void *val; void *val;
val = malloc(8 * bpf_num_possible_cpus());
skel = bpf_iter_bpf_percpu_hash_map__open(); skel = bpf_iter_bpf_percpu_hash_map__open();
if (CHECK(!skel, "bpf_iter_bpf_percpu_hash_map__open", if (CHECK(!skel, "bpf_iter_bpf_percpu_hash_map__open",
"skeleton open failed\n")) "skeleton open failed\n"))
return; return;
skel->rodata->num_cpus = bpf_num_possible_cpus(); skel->rodata->num_cpus = bpf_num_possible_cpus();
val = malloc(8 * bpf_num_possible_cpus());
err = bpf_iter_bpf_percpu_hash_map__load(skel); err = bpf_iter_bpf_percpu_hash_map__load(skel);
if (CHECK(!skel, "bpf_iter_bpf_percpu_hash_map__load", if (CHECK(!skel, "bpf_iter_bpf_percpu_hash_map__load",
...@@ -770,6 +769,7 @@ static void test_bpf_percpu_hash_map(void) ...@@ -770,6 +769,7 @@ static void test_bpf_percpu_hash_map(void)
bpf_link__destroy(link); bpf_link__destroy(link);
out: out:
bpf_iter_bpf_percpu_hash_map__destroy(skel); bpf_iter_bpf_percpu_hash_map__destroy(skel);
free(val);
} }
static void test_bpf_array_map(void) static void test_bpf_array_map(void)
...@@ -870,14 +870,13 @@ static void test_bpf_percpu_array_map(void) ...@@ -870,14 +870,13 @@ static void test_bpf_percpu_array_map(void)
void *val; void *val;
int len; int len;
val = malloc(8 * bpf_num_possible_cpus());
skel = bpf_iter_bpf_percpu_array_map__open(); skel = bpf_iter_bpf_percpu_array_map__open();
if (CHECK(!skel, "bpf_iter_bpf_percpu_array_map__open", if (CHECK(!skel, "bpf_iter_bpf_percpu_array_map__open",
"skeleton open failed\n")) "skeleton open failed\n"))
return; return;
skel->rodata->num_cpus = bpf_num_possible_cpus(); skel->rodata->num_cpus = bpf_num_possible_cpus();
val = malloc(8 * bpf_num_possible_cpus());
err = bpf_iter_bpf_percpu_array_map__load(skel); err = bpf_iter_bpf_percpu_array_map__load(skel);
if (CHECK(!skel, "bpf_iter_bpf_percpu_array_map__load", if (CHECK(!skel, "bpf_iter_bpf_percpu_array_map__load",
...@@ -933,6 +932,7 @@ static void test_bpf_percpu_array_map(void) ...@@ -933,6 +932,7 @@ static void test_bpf_percpu_array_map(void)
bpf_link__destroy(link); bpf_link__destroy(link);
out: out:
bpf_iter_bpf_percpu_array_map__destroy(skel); bpf_iter_bpf_percpu_array_map__destroy(skel);
free(val);
} }
/* An iterator program deletes all local storage in a map. */ /* An iterator program deletes all local storage in a map. */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment