2013/3/10 Chen Gang <[email protected]>: > > k is u32 which never < 0, need type cast, or cause issue. > > Signed-off-by: Chen Gang <[email protected]> > --- > arch/arm/net/bpf_jit_32.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/net/bpf_jit_32.c b/arch/arm/net/bpf_jit_32.c > index 6828ef6..a0bd8a7 100644 > --- a/arch/arm/net/bpf_jit_32.c > +++ b/arch/arm/net/bpf_jit_32.c > @@ -576,7 +576,7 @@ load_ind: > /* x = ((*(frame + k)) & 0xf) << 2; */ > ctx->seen |= SEEN_X | SEEN_DATA | SEEN_CALL; > /* the interpreter should deal with the negative K */ > - if (k < 0) > + if ((int)k < 0) > return -1; > /* offset in r1: we might have to take the slow path > */ > emit_mov_i(r_off, k, ctx); > -- > 1.7.7.6
Acked-by: Mircea Gherzan <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

