[PATCH bpf-next v11 3/9] selftests/bpf: Close fd in error path in drop_on_reuseport

[Geliang Tang]

From: Geliang Tang <tanggeli...@kylinos.cn>

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.

Acked-by: Eduard Zingerman <eddy...@gmail.com>
Signed-off-by: Geliang Tang <tanggeli...@kylinos.cn>
---
 tools/testing/selftests/bpf/prog_tests/sk_lookup.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/bpf/prog_tests/sk_lookup.c 
b/tools/testing/selftests/bpf/prog_tests/sk_lookup.c
index 597d0467a926..de2466547efe 100644
--- a/tools/testing/selftests/bpf/prog_tests/sk_lookup.c
+++ b/tools/testing/selftests/bpf/prog_tests/sk_lookup.c
@@ -994,7 +994,7 @@ static void drop_on_reuseport(const struct test *t)
 
        err = update_lookup_map(t->sock_map, SERVER_A, server1);
        if (err)
-               goto detach;
+               goto close_srv1;
 
        /* second server on destination address we should never reach */
        server2 = make_server(t->sotype, t->connect_to.ip, t->connect_to.port,
-- 
2.43.0