Re: kern/110284: [if_ethersubr] Invalid Assumption in SIOCSIFADDR in ether_ioctl()

2012-05-16 Thread Devon H. O';Dell
The following reply was made to PR kern/110284; it has been noted by GNATS. From: "Devon H. O'Dell" To: bug-follo...@freebsd.org, freebsd-net@freebsd.org, ga...@freebsd.org Cc: Subject: Re: kern/110284: [if_ethersubr] Invalid Assumption in SIOCSIFADDR in ether_ioctl() Date: W

Re: kern/110284: [if_ethersubr] Invalid Assumption in SIOCSIFADDR in ether_ioctl()

2012-05-16 Thread Devon H. O';Dell
The following reply was made to PR kern/110284; it has been noted by GNATS. From: "Devon H. O'Dell" To: bug-follo...@freebsd.org, freebsd-net@freebsd.org, ga...@freebsd.org Cc: Subject: Re: kern/110284: [if_ethersubr] Invalid Assumption in SIOCSIFADDR in ether_ioctl() Date: W

Re: kern/110284: [if_ethersubr] Invalid Assumption in SIOCSIFADDR in ether_ioctl()

2012-05-16 Thread Devon H. O';Dell
2012/5/16 Devon H. O'Dell : > Hi Gavin, freebsd-net, > > --- if_ethersubr.c      2011-09-22 20:51:37.0 -0400 > +++ /home/dho/if_ethersubr.c    2012-05-16 22:27:17.0 -0400 > @@ -1146,7 +1146,9 @@ >                switch (ifa->ifa_addr-&g

Re: kern/110284: [if_ethersubr] Invalid Assumption in SIOCSIFADDR in ether_ioctl()

2012-05-16 Thread Devon H. O';Dell
Hi Gavin, freebsd-net, I submitted this PR when I worked at Coyote Point. I'm no longer there, but I do remember this issue well. The discussion around this patch happened on IRC and not the PR, which is unfortunate. If I recall correctly, rwatson's stance was that the real fix was to fix the hein