* [PATCH net-next] bpf: fix segfault in test_verifier selftest
@ 2018-12-21 13:04 Daniel Borkmann
2018-12-21 16:57 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Daniel Borkmann @ 2018-12-21 13:04 UTC (permalink / raw)
To: davem; +Cc: ast, netdev, Daniel Borkmann
Minor fallout from merge resolution, test_verifier was segfaulting
because the REJECT result was correct, but errstr was NULL. Properly
fix it as in 339bbff2d6e0.
Fixes: 339bbff2d6e0 ("Merge git://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next")
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
---
[ Hi David, would be great if you could take it directly, thanks! ]
tools/testing/selftests/bpf/test_verifier.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/testing/selftests/bpf/test_verifier.c b/tools/testing/selftests/bpf/test_verifier.c
index baafe5c76aca..33f7d38849b8 100644
--- a/tools/testing/selftests/bpf/test_verifier.c
+++ b/tools/testing/selftests/bpf/test_verifier.c
@@ -14194,7 +14194,8 @@ static struct bpf_test tests[] = {
},
.prog_type = BPF_PROG_TYPE_SOCKET_FILTER,
.errstr_unpriv = "function calls to other bpf functions are allowed for root only",
- .result = REJECT,
+ .errstr = "!read_ok",
+ .result = REJECT,
},
{
"jset: functional",
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net-next] bpf: fix segfault in test_verifier selftest
2018-12-21 13:04 [PATCH net-next] bpf: fix segfault in test_verifier selftest Daniel Borkmann
@ 2018-12-21 16:57 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2018-12-21 16:57 UTC (permalink / raw)
To: daniel; +Cc: ast, netdev
From: Daniel Borkmann <daniel@iogearbox.net>
Date: Fri, 21 Dec 2018 14:04:46 +0100
> Minor fallout from merge resolution, test_verifier was segfaulting
> because the REJECT result was correct, but errstr was NULL. Properly
> fix it as in 339bbff2d6e0.
>
> Fixes: 339bbff2d6e0 ("Merge git://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next")
> Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
> ---
> [ Hi David, would be great if you could take it directly, thanks! ]
I'll learn how to resolve conflicts properly some day :-)
Applied, thanks Daniel.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-12-21 16:57 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-21 13:04 [PATCH net-next] bpf: fix segfault in test_verifier selftest Daniel Borkmann
2018-12-21 16:57 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).