diff options
author | Bruce M Simpson <bms@FreeBSD.org> | 2007-03-27 16:11:28 +0000 |
---|---|---|
committer | Bruce M Simpson <bms@FreeBSD.org> | 2007-03-27 16:11:28 +0000 |
commit | 75ae0c016b242351a34a5c98900ccba5364cff3e (patch) | |
tree | 0c6e5b6e3d60d1ee93994ea858c1ce89495cf428 /sys/net/if.c | |
parent | 40385a5f331dddcd7eb4a25a971f6148f5de1a68 (diff) | |
download | src-75ae0c016b242351a34a5c98900ccba5364cff3e.tar.gz src-75ae0c016b242351a34a5c98900ccba5364cff3e.zip |
Fix a case where hardware removal of an interface caused an attempt to
announce an ll_ifma which has gone away. Add a KASSERT to catch regressions.
Bug found by: Tom Uffner
Notes
Notes:
svn path=/head/; revision=167943
Diffstat (limited to 'sys/net/if.c')
-rw-r--r-- | sys/net/if.c | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/sys/net/if.c b/sys/net/if.c index 7e4df75c33c4..33adc5007978 100644 --- a/sys/net/if.c +++ b/sys/net/if.c @@ -2512,21 +2512,27 @@ if_delmulti_locked(struct ifnet *ifp, struct ifmultiaddr *ifma, int detaching) /* * If the ifnet is detaching, null out references to ifnet, * so that upper protocol layers will notice, and not attempt - * to obtain locks for an ifnet which no longer exists. - * It is OK to call rt_newmaddrmsg() with a NULL ifp. + * to obtain locks for an ifnet which no longer exists. The + * routing socket announcement must happen before the ifnet + * instance is detached from the system. */ if (detaching) { #ifdef DIAGNOSTIC printf("%s: detaching ifnet instance %p\n", __func__, ifp); #endif - ifma->ifma_ifp = NULL; + /* + * ifp may already be nulled out if we are being reentered + * to delete the ll_ifma. + */ + if (ifp != NULL) { + rt_newmaddrmsg(RTM_DELMADDR, ifma); + ifma->ifma_ifp = NULL; + } } if (--ifma->ifma_refcount > 0) return 0; - rt_newmaddrmsg(RTM_DELMADDR, ifma); - /* * If this ifma is a network-layer ifma, a link-layer ifma may * have been associated with it. Release it first if so. |