On Thu, 5 Oct 2017 19:03:05 +0200 Matteo Croce <mcr...@redhat.com> wrote:
> Commit 35e015e1f577 ("ipv6: fix net.ipv6.conf.all interface DAD handlers") > was intended to affect accept_dad flag handling in such a way that > DAD operation and mode on a given interface would be selected > according to the maximum value of conf/{all,interface}/accept_dad. > > However, addrconf_dad_begin() checks for particular cases in which we > need to skip DAD, and this check was modified in the wrong way. > > Namely, it was modified so that, if the accept_dad flag is 0 for the > given interface *or* for all interfaces, DAD would be skipped. > > We have instead to skip DAD if accept_dad is 0 for the given interface > *and* for all interfaces. > > Fixes: 35e015e1f577 ("ipv6: fix net.ipv6.conf.all interface DAD handlers") > Acked-by: Stefano Brivio <sbri...@redhat.com> > Signed-off-by: Matteo Croce <mcr...@redhat.com> This was actually: Reported-by: Erik Kline <e...@google.com> -- Stefano