Merge common code into new nd6_dad_destroy()
authorkn <kn@openbsd.org>
Fri, 6 Jan 2023 14:17:15 +0000 (14:17 +0000)
committerkn <kn@openbsd.org>
Fri, 6 Jan 2023 14:17:15 +0000 (14:17 +0000)
commit576e5f39a29cca60c6626b950b069797dd52ffc6
tree611ff3c825224aac83205e6c097e28d7ac568331
parent81b1a5f567e3c330c826411b75b8cf197bb8da4b
Merge common code into new nd6_dad_destroy()

The current code wrt. stopping DAD for and removing a particular IP from
the list is flawed.

Introduce a single nd6_dad_destroy() to the cleanup, so that there's
only one place to fix.

This is just a mechanical deduplication without significant behaviour
change;  in case a duplicated address was found, RTM_CHGADDRATTR now goes
out before cleanup, which should be no problem.

The nd6_dad_create() pendant could be done as well, but the end of
nd6_dad_start() is currently the only place where a new IP/DAD entry is
set up, so little gain besides function name symmetry.

OK claudio
sys/netinet6/nd6_nbr.c