Re: svn commit: r239353 - head/sys/netinet

2012-08-20 Thread John Baldwin
On Saturday, August 18, 2012 5:31:24 am Randall Stewart wrote: Note that in the main FreeBSD sources however, the fact that ip_input.c does *not* lock when it looks at the hash table these lines are removing from here means there is still a potential for a crash. I have a fix for this that

Re: svn commit: r239353 - head/sys/netinet

2012-08-18 Thread Randall Stewart
On Aug 17, 2012, at 8:26 AM, John Baldwin wrote: On Friday, August 17, 2012 1:51:46 am Randall Stewart wrote: Author: rrs Date: Fri Aug 17 05:51:46 2012 New Revision: 239353 URL: http://svn.freebsd.org/changeset/base/239353 Log: Ok jhb, lets move the ifa_free() down to the bottom to

Re: svn commit: r239353 - head/sys/netinet

2012-08-17 Thread John Baldwin
On Friday, August 17, 2012 1:51:46 am Randall Stewart wrote: Author: rrs Date: Fri Aug 17 05:51:46 2012 New Revision: 239353 URL: http://svn.freebsd.org/changeset/base/239353 Log: Ok jhb, lets move the ifa_free() down to the bottom to assure that *all* tables and such are removed

svn commit: r239353 - head/sys/netinet

2012-08-16 Thread Randall Stewart
Author: rrs Date: Fri Aug 17 05:51:46 2012 New Revision: 239353 URL: http://svn.freebsd.org/changeset/base/239353 Log: Ok jhb, lets move the ifa_free() down to the bottom to assure that *all* tables and such are removed before we start to free. This won't protect the Hash in ip_input.c