diff options
| author |   Prashant Bhole <bhole_prashant_q7@lab.ntt.co.jp> | 2018-03-30 09:21:00 +0900 |
|---|---|---|
| committer |   Daniel Borkmann <daniel@iogearbox.net> | 2018-03-30 22:50:15 +0200 |
| commit | 6ef6d84ceee2262a8c7a4247616c7eb71268b3f9 (patch) | |
| tree | 630eb9ec8764b825abe20e0457df291dd0c1f9eb /net/ipv4/af_inet.c | |
| parent | lib/scatterlist: add sg_init_marker() helper (diff) | |
| download | linux-dev-6ef6d84ceee2262a8c7a4247616c7eb71268b3f9.tar.xz linux-dev-6ef6d84ceee2262a8c7a4247616c7eb71268b3f9.zip | |
bpf: sockmap: initialize sg table entries properly
When CONFIG_DEBUG_SG is set, sg->sg_magic is initialized in
sg_init_table() and it is verified in sg api while navigating. We hit
BUG_ON when magic check is failed.
In functions sg_tcp_sendpage and sg_tcp_sendmsg, the struct containing
the scatterlist is already zeroed out. So to avoid extra memset, we
use sg_init_marker() to initialize sg_magic.
Fixed following things:
- In bpf_tcp_sendpage: initialize sg using sg_init_marker
- In bpf_tcp_sendmsg: Replace sg_init_table with sg_init_marker
- In bpf_tcp_push: Replace memset with sg_init_table where consumed
sg entry needs to be re-initialized.
Signed-off-by: Prashant Bhole <bhole_prashant_q7@lab.ntt.co.jp>
Acked-by: John Fastabend <john.fastabend@gmail.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Diffstat (limited to 'net/ipv4/af_inet.c')
0 files changed, 0 insertions, 0 deletions