Date: Fri, 14 Aug 2009 10:09:45 +0000 (UTC) From: Julian Elischer <julian@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/netinet/ipfw ip_fw2.c Message-ID: <200908141010.n7EAA16m051863@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
julian 2009-08-14 10:09:45 UTC
FreeBSD src repository
Modified files:
sys/netinet/ipfw ip_fw2.c
Log:
SVN rev 196201 on 2009-08-14 10:09:45Z by julian
Fix ipfw crash on uid or gid check.
Receiving any ip packet for which there is no existing socket will
crash if ipfw has a uid or gid test rule, as the uid/gid
of the non existent owner of said non existent socket is tested.
Brooks introduced this error as part of his >16 gids patch.
It appears to be a cut-n-paste error from similar code a few lines
before. The old code used the 'pcb' variable here, but in the
new code that switched the 'inp' variable, which is often NULL
and what is tested in the code further up. The rest of the multi-gid
patch for ipfw seems solid (and cleaner than previous code).
Reviewed by: brooks
Approved by: re (rwatson)
Revision Changes Path
1.12 +1 -1 src/sys/netinet/ipfw/ip_fw2.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200908141010.n7EAA16m051863>