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 t

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 th

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 remov

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 but