diff options
| author | Stanislav Fomichev <[email protected]> | 2019-04-11 15:53:16 -0700 |
|---|---|---|
| committer | Daniel Borkmann <[email protected]> | 2019-04-16 09:51:48 +0200 |
| commit | bcbccad694b7ef0de9a993ecd918231c10a1496a (patch) | |
| tree | 0997fe87eaf8225cf182bd12dd0f3ef270d7102a | |
| parent | efb2ddc4ce5dba9b6c5ec106528d18a645424f3f (diff) | |
selftests/bpf: bring back (void *) cast to set_ipv4_csum in test_tc_tunnel
It was removed in commit 166b5a7f2ca3 ("selftests_bpf: extend
test_tc_tunnel for UDP encap") without any explanation.
Otherwise I see:
progs/test_tc_tunnel.c:160:17: warning: taking address of packed member 'ip' of class or structure
'v4hdr' may result in an unaligned pointer value [-Waddress-of-packed-member]
set_ipv4_csum(&h_outer.ip);
^~~~~~~~~~
1 warning generated.
Cc: Alan Maguire <[email protected]>
Cc: Willem de Bruijn <[email protected]>
Fixes: 166b5a7f2ca3 ("selftests_bpf: extend test_tc_tunnel for UDP encap")
Signed-off-by: Stanislav Fomichev <[email protected]>
Acked-by: Song Liu <[email protected]>
Reviewed-by: Alan Maguire <[email protected]>
Signed-off-by: Daniel Borkmann <[email protected]>
| -rw-r--r-- | tools/testing/selftests/bpf/progs/test_tc_tunnel.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/bpf/progs/test_tc_tunnel.c b/tools/testing/selftests/bpf/progs/test_tc_tunnel.c index bcb00d737e95..ab56a6a72b7a 100644 --- a/tools/testing/selftests/bpf/progs/test_tc_tunnel.c +++ b/tools/testing/selftests/bpf/progs/test_tc_tunnel.c @@ -157,7 +157,7 @@ static __always_inline int encap_ipv4(struct __sk_buff *skb, __u8 encap_proto, bpf_ntohs(h_outer.ip.tot_len)); h_outer.ip.protocol = encap_proto; - set_ipv4_csum(&h_outer.ip); + set_ipv4_csum((void *)&h_outer.ip); /* store new outer network header */ if (bpf_skb_store_bytes(skb, ETH_HLEN, &h_outer, olen, |