svn commit: r327112 - head/sys/netinet6
Alexander Kabaev
kan at FreeBSD.org
Sat Dec 23 16:45:28 UTC 2017
Author: kan
Date: Sat Dec 23 16:45:26 2017
New Revision: 327112
URL: https://svnweb.freebsd.org/changeset/base/327112
Log:
Silence clang analyzer false positive.
clang does not know that two lookup calls will return the same
pointer, so it assumes correctly that using the old pointer
after dropping the reference to it is a bit risky.
Modified:
head/sys/netinet6/nd6_nbr.c
Modified: head/sys/netinet6/nd6_nbr.c
==============================================================================
--- head/sys/netinet6/nd6_nbr.c Sat Dec 23 16:45:24 2017 (r327111)
+++ head/sys/netinet6/nd6_nbr.c Sat Dec 23 16:45:26 2017 (r327112)
@@ -1307,7 +1307,8 @@ nd6_dad_stop(struct ifaddr *ifa)
* we were waiting for it to stop, so re-do the lookup.
*/
nd6_dad_rele(dp);
- if (nd6_dad_find(ifa, NULL) == NULL)
+ dp = nd6_dad_find(ifa, NULL);
+ if (dp == NULL)
return;
nd6_dad_del(dp);
More information about the svn-src-head
mailing list