Date: Mon, 30 Nov 1998 23:14:06 -0800 From: David Greenman <dg@root.com> To: Matthew Dillon <dillon@apollo.backplane.com> Cc: freebsd-current@FreeBSD.ORG Subject: Re: D.O.S. attack protection enhancements commit (ICMP_BANDLIM) Message-ID: <199812010714.XAA26714@root.com> In-Reply-To: Your message of "Mon, 30 Nov 1998 22:07:47 PST." <199812010607.WAA03051@apollo.backplane.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> I would like to make a commit to -current similar to the following > (the one below is actually relative to -stable, but we are in final > testing of the same patch to our -current tree). > > The commit I want to commit is just the ICMP_BANDLIM section. I am The only comment I have at the moment is that the patches need a little KNF work. The mixing of indentation amounts within a function is particularly annoying and makes the code difficult to read. Indents should be tabs, with half tabs (4 spaces) used on continued lines (such as if () conditionals). The patches have other KNF problems as well...see 'man 9 style'. In principle, I'm not against these sorts of changes, but I tend to think that a more general scheme implemented perhaps inside the ipfw framework would be more appropriate. I also generally like to avoid compile time options for things like this, but I"m sympathetic for performance reducing enhancements. -DG David Greenman Co-founder/Principal Architect, The FreeBSD Project To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199812010714.XAA26714>