On Mon, Jan 28, 2019 at 02:50:34PM -0600, Willem de Bruijn wrote: > On Mon, Jan 28, 2019 at 2:53 AM Steffen Klassert > <steffen.klass...@secunet.com> wrote: > > + > > + if (skb_needs_linearize(skb, features) && > > + __skb_linearize(skb)) { > > + skb->next = NULL; > > + kfree_skb_list(skb->next); > > inverse order
Oh yes, apparently. > > also, I would probably deduplicate with the same branch above in a new > err_linearize: block Will do that. Thanks for the review!