On Mon, Aug 22, 2016 at 06:59:51PM +0800, f...@ikuai8.com wrote: > From: Gao Feng <f...@ikuai8.com> > > Use PPP_ALLSTATIONS, PPP_UI, and SEND_SHUTDOWN instead of 0xff, > 0x03, and 2 separately. > Apart from the checkpatch errors,
Acked-by: Guillaume Nault <g.na...@alphalink.fr> > @@ -325,8 +324,8 @@ static int pppol2tp_sendmsg(struct socket *sock, struct > msghdr *m, > skb_reserve(skb, uhlen); > > /* Add PPP header */ > - skb->data[0] = ppph[0]; > - skb->data[1] = ppph[1]; > + skb->data[0] = PPP_ALLSTATIONS; > Trailing whitespace. > @@ -398,14 +396,14 @@ static int pppol2tp_xmit(struct ppp_channel *chan, > struct sk_buff *skb) > sizeof(struct iphdr) + /* IP header */ > uhlen + /* UDP header (if L2TP_ENCAPTYPE_UDP) */ > session->hdr_len + /* L2TP header */ > - sizeof(ppph); /* PPP header */ > + 2; /* 2 bytes for PPP_ALLSTATIONS & PPP_UI > */ > if (skb_cow_head(skb, headroom)) > goto abort_put_sess_tun; > > /* Setup PPP header */ > - __skb_push(skb, sizeof(ppph)); > - skb->data[0] = ppph[0]; > - skb->data[1] = ppph[1]; > + __skb_push(skb, 2); > Here too.