On Tue, Jun 14, 2011 at 10:23:36PM +0200, Claudio Jeker wrote: > On Wed, Jun 08, 2011 at 09:53:24AM +0200, Claudio Jeker wrote: > > Next step on my quest to unify pf_test and pf_test6. > > Move the fragment handling and some other protocol specific tests into > > pf_setup_pdesc(). IPv6 already does this mostly but only IPv4 did not. So > > this diff brings that more into sync. It also includes some additional > > cleanups. > > > > Works for me but needs some more testing. > > New version with fixes found by bluhm@ (mostly make sure that the right > header and mbufs are used in all cases). > > -- > :wq Claudio
Blows up REAL good on my 6xamd64 box. Within a minute or two. I put some debug in and pf_scrub() is being passed a NULL 'm' parameter from pf_test(). Hand transcribed trace: pf_scrub()+0x12 pf_test()+0x93b ipv4_input()+0x22a ipintr()+0x51 netintr()+0xda softintr_dispatch()+0x5d Xsoftnet()+0x2d This box is my nfs server and is using the default pf.conf file as far as I know. Also blew up on my firewall as soon as traffic started transitting it. Same trace as far as I can see. .... Ken