From: Maxim Mikityanskiy <[email protected]>

The u64_offset_to_skb_data test is supposed to make a 64-bit fill, but
instead makes a 16-bit one. Fix the test according to its intention. The
16-bit fill is covered by u16_offset_to_skb_data.

Signed-off-by: Maxim Mikityanskiy <[email protected]>
---
 tools/testing/selftests/bpf/progs/verifier_spill_fill.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/bpf/progs/verifier_spill_fill.c 
b/tools/testing/selftests/bpf/progs/verifier_spill_fill.c
index 39fe3372e0e0..84eccab36582 100644
--- a/tools/testing/selftests/bpf/progs/verifier_spill_fill.c
+++ b/tools/testing/selftests/bpf/progs/verifier_spill_fill.c
@@ -243,7 +243,7 @@ l0_%=:      r0 = 0;                                         
\
 
 SEC("tc")
 __description("Spill u32 const scalars.  Refill as u64.  Offset to skb->data")
-__failure __msg("invalid access to packet")
+__failure __msg("math between pkt pointer and register with unbounded min 
value is not allowed")
 __naked void u64_offset_to_skb_data(void)
 {
        asm volatile ("                                 \
@@ -253,7 +253,7 @@ __naked void u64_offset_to_skb_data(void)
        w7 = 20;                                        \
        *(u32*)(r10 - 4) = r6;                          \
        *(u32*)(r10 - 8) = r7;                          \
-       r4 = *(u16*)(r10 - 8);                          \
+       r4 = *(u64*)(r10 - 8);                          \
        r0 = r2;                                        \
        /* r0 += r4 R0=pkt R2=pkt R3=pkt_end R4=umax=65535 */\
        r0 += r4;                                       \
-- 
2.42.1


Reply via email to