lkml.org 
[lkml]   [2019]   [Apr]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 27/69] vrf: check accept_source_route on the original netdevice
    Date
    From: Stephen Suryaputra <ssuryaextr@gmail.com>

    [ Upstream commit 8c83f2df9c6578ea4c5b940d8238ad8a41b87e9e ]

    Configuration check to accept source route IP options should be made on
    the incoming netdevice when the skb->dev is an l3mdev master. The route
    lookup for the source route next hop also needs the incoming netdev.

    v2->v3:
    - Simplify by passing the original netdevice down the stack (per David
    Ahern).

    Signed-off-by: Stephen Suryaputra <ssuryaextr@gmail.com>
    Reviewed-by: David Ahern <dsahern@gmail.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    include/net/ip.h | 2 +-
    net/ipv4/ip_input.c | 7 +++----
    net/ipv4/ip_options.c | 4 ++--
    3 files changed, 6 insertions(+), 7 deletions(-)

    --- a/include/net/ip.h
    +++ b/include/net/ip.h
    @@ -603,7 +603,7 @@ int ip_options_get_from_user(struct net
    unsigned char __user *data, int optlen);
    void ip_options_undo(struct ip_options *opt);
    void ip_forward_options(struct sk_buff *skb);
    -int ip_options_rcv_srr(struct sk_buff *skb);
    +int ip_options_rcv_srr(struct sk_buff *skb, struct net_device *dev);

    /*
    * Functions provided by ip_sockglue.c
    --- a/net/ipv4/ip_input.c
    +++ b/net/ipv4/ip_input.c
    @@ -259,11 +259,10 @@ int ip_local_deliver(struct sk_buff *skb
    ip_local_deliver_finish);
    }

    -static inline bool ip_rcv_options(struct sk_buff *skb)
    +static inline bool ip_rcv_options(struct sk_buff *skb, struct net_device *dev)
    {
    struct ip_options *opt;
    const struct iphdr *iph;
    - struct net_device *dev = skb->dev;

    /* It looks as overkill, because not all
    IP options require packet mangling.
    @@ -299,7 +298,7 @@ static inline bool ip_rcv_options(struct
    }
    }

    - if (ip_options_rcv_srr(skb))
    + if (ip_options_rcv_srr(skb, dev))
    goto drop;
    }

    @@ -362,7 +361,7 @@ static int ip_rcv_finish(struct net *net
    }
    #endif

    - if (iph->ihl > 5 && ip_rcv_options(skb))
    + if (iph->ihl > 5 && ip_rcv_options(skb, dev))
    goto drop;

    rt = skb_rtable(skb);
    --- a/net/ipv4/ip_options.c
    +++ b/net/ipv4/ip_options.c
    @@ -612,7 +612,7 @@ void ip_forward_options(struct sk_buff *
    }
    }

    -int ip_options_rcv_srr(struct sk_buff *skb)
    +int ip_options_rcv_srr(struct sk_buff *skb, struct net_device *dev)
    {
    struct ip_options *opt = &(IPCB(skb)->opt);
    int srrspace, srrptr;
    @@ -647,7 +647,7 @@ int ip_options_rcv_srr(struct sk_buff *s

    orefdst = skb->_skb_refdst;
    skb_dst_set(skb, NULL);
    - err = ip_route_input(skb, nexthop, iph->saddr, iph->tos, skb->dev);
    + err = ip_route_input(skb, nexthop, iph->saddr, iph->tos, dev);
    rt2 = skb_rtable(skb);
    if (err || (rt2->rt_type != RTN_UNICAST && rt2->rt_type != RTN_LOCAL)) {
    skb_dst_drop(skb);

    \
     
     \ /
      Last update: 2019-04-15 21:03    [W:4.215 / U:0.124 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site