Re: [PATCH net v2] skbuff: fix compilation warnings in skb_dump()

From: Willem de Bruijn
Date: Tue Jul 16 2019 - 15:01:47 EST


On Tue, Jul 16, 2019 at 6:53 PM Nathan Chancellor
<natechancellor@xxxxxxxxx> wrote:
>
> On Tue, Jul 16, 2019 at 11:43:05AM -0400, Qian Cai wrote:
> > The commit 6413139dfc64 ("skbuff: increase verbosity when dumping skb
> > data") introduced a few compilation warnings.
> >
> > net/core/skbuff.c:766:32: warning: format specifies type 'unsigned
> > short' but the argument has type 'unsigned int' [-Wformat]
> > level, sk->sk_family, sk->sk_type,
> > sk->sk_protocol);
> > ^~~~~~~~~~~
> > net/core/skbuff.c:766:45: warning: format specifies type 'unsigned
> > short' but the argument has type 'unsigned int' [-Wformat]
> > level, sk->sk_family, sk->sk_type,
> > sk->sk_protocol);
> > ^~~~~~~~~~~~~~~
> >
> > Fix them by using the proper types.
> >
> > Fixes: 6413139dfc64 ("skbuff: increase verbosity when dumping skb data")
> > Signed-off-by: Qian Cai <cai@xxxxxx>
>
> Reviewed-by: Nathan Chancellor <natechancellor@xxxxxxxxx>

Acked-by: Willem de Bruijn <willemb@xxxxxxxxxx>

Thanks Qian.