> -----Original Message-----
> From: Stephen Hemminger [mailto:[email protected]]
> Sent: Thursday, January 17, 2019 1:33 AM
> To: wangyunjian <[email protected]>
> Cc: [email protected]; xudingke <[email protected]>;
> [email protected]; Nkolay Aleksandrov
> <[email protected]>; Roopa Prabhu
> <[email protected]>
> Subject: Re: [PATCH net v3] net: bridge: Fix ethernet header pointer before
> check skb forwardable
>
> On Wed, 16 Jan 2019 21:04:21 +0800
> wangyunjian <[email protected]> wrote:
>
> > if (unlikely(netpoll_tx_running(to->br->dev))) {
> > + skb_push(skb, ETH_HLEN);
> > if (!is_skb_forwardable(skb->dev, skb)) {
> > kfree_skb(skb);
> > } else {
> > - skb_push(skb, ETH_HLEN);
> > br_netpoll_send_skb(to, skb);
> > }
> > return;
>
> The patch looks correct.
>
> On minor style issue is that after moving skb_push out of the if statement,
> both branches of the statement now contain a single line and therefore the
> brackets are no longer necessary. Suprised that checkpatch did not complain
> about this.
OK, I'll fix it and send the v4 later.
Thanks
Yunjian
