cvs commit: src/sys/netinet tcp_subr.c

Robert Watson rwatson at FreeBSD.org
Tue Nov 23 08:23:14 PST 2004


rwatson     2004-11-23 16:23:13 UTC

  FreeBSD src repository

  Modified files:
    sys/netinet          tcp_subr.c 
  Log:
  Assert the inpcb lock in tcp_twstart(), which does both read-modify-write
  on the tcpcb, but also calls into tcp_close() and tcp_twrespond().
  
  Annotate that tcp_twrecycleable() requires the inpcb lock because it does
  a series of non-atomic reads of the tcpcb, but is currently called
  without the inpcb lock by the caller.  This is a bug.
  
  Assert the inpcb lock in tcp_twclose() as it performs a read-modify-write
  of the timewait structure/inpcb, and calls in_pcbdetach() which requires
  the lock.
  
  Assert the inpcb lock in tcp_twrespond(), as it performs multiple
  non-atomic reads of the tcptw and inpcb structures, as well as calling
  mac_create_mbuf_from_inpcb(), tcpip_fillheaders(), which require the
  inpcb lock.
  
  MFC after:      2 weeks
  
  Revision  Changes    Path
  1.210     +10 -0     src/sys/netinet/tcp_subr.c


More information about the cvs-src mailing list