Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 22 Jun 2014 16:32:41 +0000 (UTC)
From:      Michael Tuexen <tuexen@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   svn commit: r267725 - stable/10/sys/netinet
Message-ID:  <201406221632.s5MGWfhn039218@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: tuexen
Date: Sun Jun 22 16:32:41 2014
New Revision: 267725
URL: http://svnweb.freebsd.org/changeset/base/267725

Log:
  MFC r263922:
  
  Handle an edge case of address management similar to TCP.
  This needs to be reconsidered when the address handling
  will be reimplemented.
  The patch is from rrs@.

Modified:
  stable/10/sys/netinet/sctp_pcb.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/netinet/sctp_pcb.c
==============================================================================
--- stable/10/sys/netinet/sctp_pcb.c	Sun Jun 22 16:30:52 2014	(r267724)
+++ stable/10/sys/netinet/sctp_pcb.c	Sun Jun 22 16:32:41 2014	(r267725)
@@ -772,7 +772,14 @@ sctp_del_addr_from_vrf(uint32_t vrf_id, 
 		}
 		SCTPDBG(SCTP_DEBUG_PCB4, "Deleting ifa %p\n", (void *)sctp_ifap);
 		sctp_ifap->localifa_flags &= SCTP_ADDR_VALID;
-		sctp_ifap->localifa_flags |= SCTP_BEING_DELETED;
+		/*
+		 * We don't set the flag. This means that the structure will
+		 * hang around in EP's that have bound specific to it until
+		 * they close. This gives us TCP like behavior if someone
+		 * removes an address (or for that matter adds it right
+		 * back).
+		 */
+		/* sctp_ifap->localifa_flags |= SCTP_BEING_DELETED; */
 		vrf->total_ifa_count--;
 		LIST_REMOVE(sctp_ifap, next_bucket);
 		sctp_remove_ifa_from_ifn(sctp_ifap);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201406221632.s5MGWfhn039218>