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.

Reply via email to