lkml.org 
[lkml]   [2017]   [Oct]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Date
    Subject[PATCH 3.2 29/74] ipv6: dad: don't remove dynamic addresses if link is down
    3.2.94-rc1 review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Sabrina Dubroca <sd@queasysnail.net>

    commit ec8add2a4c9df723c94a863b8fcd6d93c472deed upstream.

    Currently, when the link for $DEV is down, this command succeeds but the
    address is removed immediately by DAD (1):

    ip addr add 1111::12/64 dev $DEV valid_lft 3600 preferred_lft 1800

    In the same situation, this will succeed and not remove the address (2):

    ip addr add 1111::12/64 dev $DEV
    ip addr change 1111::12/64 dev $DEV valid_lft 3600 preferred_lft 1800

    The comment in addrconf_dad_begin() when !IF_READY makes it look like
    this is the intended behavior, but doesn't explain why:

    * If the device is not ready:
    * - keep it tentative if it is a permanent address.
    * - otherwise, kill it.

    We clearly cannot prevent userspace from doing (2), but we can make (1)
    work consistently with (2).

    addrconf_dad_stop() is only called in two cases: if DAD failed, or to
    skip DAD when the link is down. In that second case, the fix is to avoid
    deleting the address, like we already do for permanent addresses.

    Fixes: 3c21edbd1137 ("[IPV6]: Defer IPv6 device initialization until the link becomes ready.")
    Signed-off-by: Sabrina Dubroca <sd@queasysnail.net>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    [bwh: Backported to 3.2: adjust context]
    Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
    ---
    net/ipv6/addrconf.c | 18 +++++++++---------
    1 file changed, 9 insertions(+), 9 deletions(-)

    --- a/net/ipv6/addrconf.c
    +++ b/net/ipv6/addrconf.c
    @@ -1385,16 +1385,8 @@ static void addrconf_dad_stop(struct ine
    if (dad_failed)
    ifp->flags |= IFA_F_DADFAILED;

    - if (ifp->flags&IFA_F_PERMANENT) {
    - spin_lock_bh(&ifp->lock);
    - addrconf_del_timer(ifp);
    - ifp->flags |= IFA_F_TENTATIVE;
    - spin_unlock_bh(&ifp->lock);
    - if (dad_failed)
    - ipv6_ifa_notify(0, ifp);
    - in6_ifa_put(ifp);
    #ifdef CONFIG_IPV6_PRIVACY
    - } else if (ifp->flags&IFA_F_TEMPORARY) {
    + if (ifp->flags&IFA_F_TEMPORARY) {
    struct inet6_ifaddr *ifpub;
    spin_lock_bh(&ifp->lock);
    ifpub = ifp->ifpub;
    @@ -1407,7 +1399,16 @@ static void addrconf_dad_stop(struct ine
    spin_unlock_bh(&ifp->lock);
    }
    ipv6_del_addr(ifp);
    + } else
    #endif
    + if (ifp->flags&IFA_F_PERMANENT || !dad_failed) {
    + spin_lock_bh(&ifp->lock);
    + addrconf_del_timer(ifp);
    + ifp->flags |= IFA_F_TENTATIVE;
    + spin_unlock_bh(&ifp->lock);
    + if (dad_failed)
    + ipv6_ifa_notify(0, ifp);
    + in6_ifa_put(ifp);
    } else
    ipv6_del_addr(ifp);
    }
    \
     
     \ /
      Last update: 2017-10-09 15:01    [W:4.390 / U:0.616 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site