kern/137164: [netinet] [patch] assert panic imo_match_source()

Stef Walter stef at
Tue Oct 6 15:40:04 UTC 2009

The following reply was made to PR kern/137164; it has been noted by GNATS.

From: Stef Walter <stef at>
To: bug-followup at, jhanna at, 
 Bruce Simpson <bms at>
Subject: Re: kern/137164: [netinet] [patch] assert panic imo_match_source()
Date: Tue, 06 Oct 2009 10:37:31 -0500

 This is a multi-part message in MIME format.
 Content-Type: text/plain; charset=ISO-8859-1
 Content-Transfer-Encoding: 7bit
 Thanks for working on getting all these multicast fixes in. Much
 Just one more thing, previous to 7.x FreeBSD would return EADDRINUSE
 in the case of a double IP_ADD_MEMBERSHIP. Software like quagga uses
 this error code to detect this condition.
 As patched (and MFC'd in 7.x and 8.x) EINVAL is returned instead and
 this confuses such software.
 Currently the multicast code does not remove memberships from its
 internal structures when an interface goes down. It's hard for userland
 to reliably track the condition of a multicast membership that didn't go
 away due to an interface going down. So software like quagga uses
 EADDRINUSE to track the condition.
 Obviously, as you Bruce mentioned, an better solution needs to be found
 eventually WRT to 'dynamic' interfaces and the multicast code. But for
 now would the attached patch be acceptable? It would prevent regressions
 from FreeBSD 6.x.
 Thanks for considering,
 Content-Type: text/x-diff;
 Content-Transfer-Encoding: 7bit
 Content-Disposition: inline;
 --- sys/netinet/in_mcast.c.orig	2009-09-30 16:43:35.000000000 +0000
 +++ sys/netinet/in_mcast.c	2009-09-30 17:04:59.000000000 +0000
 @@ -2010,5 +2010,5 @@
  			 * is atomic with allocation of a membership.
 -			error = EINVAL;
 +			error = EADDRINUSE;
  			goto out_inp_locked;

More information about the freebsd-net mailing list