DragonFly BSD
DragonFly commits List (threaded) for 2010-09
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

Re: git: network - Remove crit_exit/crit_enter wrappers in pf.c


From: Matthew Dillon <dillon@xxxxxxxxxxxxxxxxxxxx>
Date: Fri, 17 Sep 2010 17:54:39 -0700 (PDT)

:While looking at this change, I found an odd piece of code in pf_route6()
:at line 5384; there's no prior call to crit_exit() in this function.
:Is this OK?
:
:	/* Cheat. XXX why only in the v6 case??? */
:	if (r->rt == PF_FASTROUTE) {
:		m0->m_pkthdr.pf.flags |= PF_TAG_GENERATED;
:		ip6_output(m0, NULL, NULL, 0, NULL, NULL, NULL);
:		crit_enter();
:		return;
:	}

    No, that looks totaly broken.  I'll remove the bogus enter.
    Nice catch.

					-Matt
					Matthew Dillon 
					<dillon@backplane.com>



[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]