Commit 03e982ee authored by Lawrence Brakmo's avatar Lawrence Brakmo Committed by David S. Miller

bpf: Fix tcp_clamp_kern.c sample program

The program was returning -1 in some cases which is not allowed
by the verifier any longer.

Fixes: 390ee7e2 ("bpf: enforce return code for cgroup-bpf programs")
Signed-off-by: default avatarLawrence Brakmo <brakmo@fb.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent e1853319
...@@ -41,8 +41,10 @@ int bpf_clamp(struct bpf_sock_ops *skops) ...@@ -41,8 +41,10 @@ int bpf_clamp(struct bpf_sock_ops *skops)
/* For testing purposes, only execute rest of BPF program /* For testing purposes, only execute rest of BPF program
* if neither port numberis 55601 * if neither port numberis 55601
*/ */
if (bpf_ntohl(skops->remote_port) != 55601 && skops->local_port != 55601) if (bpf_ntohl(skops->remote_port) != 55601 && skops->local_port != 55601) {
return -1; skops->reply = -1;
return 0;
}
op = (int) skops->op; op = (int) skops->op;
...@@ -66,9 +68,9 @@ int bpf_clamp(struct bpf_sock_ops *skops) ...@@ -66,9 +68,9 @@ int bpf_clamp(struct bpf_sock_ops *skops)
/* Set sndbuf and rcvbuf of active connections */ /* Set sndbuf and rcvbuf of active connections */
rv = bpf_setsockopt(skops, SOL_SOCKET, SO_SNDBUF, rv = bpf_setsockopt(skops, SOL_SOCKET, SO_SNDBUF,
&bufsize, sizeof(bufsize)); &bufsize, sizeof(bufsize));
rv = rv*100 + bpf_setsockopt(skops, SOL_SOCKET, rv += bpf_setsockopt(skops, SOL_SOCKET,
SO_RCVBUF, &bufsize, SO_RCVBUF, &bufsize,
sizeof(bufsize)); sizeof(bufsize));
break; break;
case BPF_SOCK_OPS_ACTIVE_ESTABLISHED_CB: case BPF_SOCK_OPS_ACTIVE_ESTABLISHED_CB:
rv = bpf_setsockopt(skops, SOL_TCP, rv = bpf_setsockopt(skops, SOL_TCP,
...@@ -80,12 +82,12 @@ int bpf_clamp(struct bpf_sock_ops *skops) ...@@ -80,12 +82,12 @@ int bpf_clamp(struct bpf_sock_ops *skops)
rv = bpf_setsockopt(skops, SOL_TCP, rv = bpf_setsockopt(skops, SOL_TCP,
TCP_BPF_SNDCWND_CLAMP, TCP_BPF_SNDCWND_CLAMP,
&clamp, sizeof(clamp)); &clamp, sizeof(clamp));
rv = rv*100 + bpf_setsockopt(skops, SOL_SOCKET, rv += bpf_setsockopt(skops, SOL_SOCKET,
SO_SNDBUF, &bufsize, SO_SNDBUF, &bufsize,
sizeof(bufsize)); sizeof(bufsize));
rv = rv*100 + bpf_setsockopt(skops, SOL_SOCKET, rv += bpf_setsockopt(skops, SOL_SOCKET,
SO_RCVBUF, &bufsize, SO_RCVBUF, &bufsize,
sizeof(bufsize)); sizeof(bufsize));
break; break;
default: default:
rv = -1; rv = -1;
......
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