summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLawrence Brakmo <brakmo@fb.com>2017-11-11 07:19:53 +0100
committerDavid S. Miller <davem@davemloft.net>2017-11-11 07:52:41 +0100
commit2ff969fbe2bfa4486b66226917352d4bb12ec1cb (patch)
tree264456174e35e9f3b3ec3259774814c3e3367723
parentbpf: Fix tcp_bufs_kern.c sample program (diff)
downloadlinux-2ff969fbe2bfa4486b66226917352d4bb12ec1cb.tar.xz
linux-2ff969fbe2bfa4486b66226917352d4bb12ec1cb.zip
bpf: Fix tcp_cong_kern.c sample program
The program was returning -1 in some cases which is not allowed by the verifier any longer. Fixes: 390ee7e29fc8 ("bpf: enforce return code for cgroup-bpf programs") Signed-off-by: Lawrence Brakmo <brakmo@fb.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--samples/bpf/tcp_cong_kern.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/samples/bpf/tcp_cong_kern.c b/samples/bpf/tcp_cong_kern.c
index dac15bce1fa9..ad0f1ba8206a 100644
--- a/samples/bpf/tcp_cong_kern.c
+++ b/samples/bpf/tcp_cong_kern.c
@@ -39,8 +39,10 @@ int bpf_cong(struct bpf_sock_ops *skops)
* if neither port numberis 55601
*/
if (bpf_ntohl(skops->remote_port) != 55601 &&
- skops->local_port != 55601)
- return -1;
+ skops->local_port != 55601) {
+ skops->reply = -1;
+ return 1;
+ }
op = (int) skops->op;