On Thu, Sep 29, 2016 at 4:03 AM, Shmulik Ladkani <shmulik.ladk...@gmail.com> wrote: > skb2->skb_iif = skb->dev->ifindex; > skb2->dev = dev; > - err = dev_queue_xmit(skb2); > + if (tcf_mirred_act_direction(m_eaction) & AT_EGRESS) > + err = dev_queue_xmit(skb2); > + else > + netif_receive_skb(skb2);
Any reason why not check the return value here?