svn commit: r239353 - head/sys/netinet

John Baldwin jhb at freebsd.org
Fri Aug 17 14:28:12 UTC 2012


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 before
>   we start to free. This won't protect the Hash in ip_input.c
>   but in theory should protect any other uses that *do* use locks.

Eh, this is just a nop unless there is a reference counting bug.  Only the 
last reference would free the memory if the reference count is correct, so 
this is just adding obfuscation by moving the ifa_free() away from the 
associated TAILQ_REMOVE().

-- 
John Baldwin


More information about the svn-src-all mailing list