Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 7 May 2002 15:18:45 -0700 (PDT)
From:      Kelly Yancey <kbyanc@posi.net>
To:        Kelly Yancey <kbyanc@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, <cvs-all@FreeBSD.org>
Subject:   Re: cvs commit: src/sys/netinet if_ether.c src/sys/net if_dl.h     if_iso88025subr.c iso88025.h
Message-ID:  <20020507151754.Q28143-100000@gateway.posi.net>
In-Reply-To: <200205072214.g47ME6u60398@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 7 May 2002, Kelly Yancey wrote:

> kbyanc      2002/05/07 15:14:06 PDT
>
>   Modified files:
>     sys/netinet          if_ether.c
>     sys/net              if_dl.h if_iso88025subr.c iso88025.h
>   Log:
>   Move ISO88025 source routing information into sockaddr_dl's sdl_data
>   field.  This returns the sdl_data field to a variable-length field.  More
>   importantly, this prevents a easily-reproduceable data-corruption bug when
>   the interface name plus the hardware address exceed the sdl_data field's
>   original 12 byte limit.  However, token-ring interfaces may still overflow
>   the new sdl_data field's 46 byte limit if the interface name exceeds 6
>   characters (since 6 characters for interface name plus 6 for hardware
>   address plus 34 for source routing = the size of sdl_data).  Further
>   refinements could overcome this limitation but would break binary
>   compatibility; this commit only addresses fixing the bug for
>   commonly-occuring cases without breaking binary compatibility with the
>   intention that the functionality can be MFC'ed to -stable.
>
>     See message ID's (both send to -arch):
>           20020421013332.F87395-100000@gateway.posi.net
>           20020430181359.G11009-300000@gateway.posi.net
>     for a more thorough description of the bug addressed and how to
>   reproduce it.
>
>   Approved by:    silence on -arch and -net
>   Sponsored by:   NTT Multimedia Communications Labs
>   MFC after:      1 week
		    (pending re's approval, of course)

  Kelly
  kbyanc@{posi.net,FreeBSD.org}


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




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