mirror of
https://github.com/torvalds/linux.git
synced 2026-06-04 20:46:48 +02:00
selftests/bpf: Close fd in error path in drop_on_reuseport
In the error path when update_lookup_map() fails in drop_on_reuseport in
prog_tests/sk_lookup.c, "server1", the fd of server 1, should be closed.
This patch fixes this by using "goto close_srv1" lable instead of "detach"
to close "server1" in this case.
Fixes: 0ab5539f85 ("selftests/bpf: Tests for BPF_SK_LOOKUP attach point")
Acked-by: Eduard Zingerman <eddyz87@gmail.com>
Signed-off-by: Geliang Tang <tanggeliang@kylinos.cn>
Link: https://lore.kernel.org/r/86aed33b4b0ea3f04497c757845cff7e8e621a2d.1720515893.git.tanggeliang@kylinos.cn
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>
This commit is contained in:
parent
7046345d48
commit
adae187ebe
|
|
@ -994,7 +994,7 @@ static void drop_on_reuseport(const struct test *t)
|
||||||
|
|
||||||
err = update_lookup_map(t->sock_map, SERVER_A, server1);
|
err = update_lookup_map(t->sock_map, SERVER_A, server1);
|
||||||
if (err)
|
if (err)
|
||||||
goto detach;
|
goto close_srv1;
|
||||||
|
|
||||||
/* second server on destination address we should never reach */
|
/* second server on destination address we should never reach */
|
||||||
server2 = make_server(t->sotype, t->connect_to.ip, t->connect_to.port,
|
server2 = make_server(t->sotype, t->connect_to.ip, t->connect_to.port,
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user