Firewall Wizards mailing list archives
Re: FWTK and smap/smapd
From: "Marcus J. Ranum" <mjr () ranum com>
Date: Thu, 18 Jul 2002 17:46:38 -0400
Joseph S D Yao wrote:
However, my major contribution to the consolidated patch - besides removing a lot of bugs - was to put in a lot of comments. "It is obvious" that very little code is self-explanatory. ;-)
Security critical code shouldn't be commented. :) It should either be sufficiently obvious or the auditor should be sufficiently skilled that comments aren't needed -- besides they just serve as distractions. :) If you don't have comments, your comments and your code are never in disagreement!!! :) mjr. --- Marcus J. Ranum - Computer and communications Security Expertise mjr () ranum com (http://www.ranum.com) _______________________________________________ firewall-wizards mailing list firewall-wizards () honor icsalabs com http://honor.icsalabs.com/mailman/listinfo/firewall-wizards
Current thread:
- FWTK and smap/smapd Behm, Jeffrey L. (Jul 16)
- Re: FWTK and smap/smapd Jerry Wintrode (Jul 16)
- Re: FWTK and smap/smapd Kevin Steves (Jul 16)
- Re: FWTK and smap/smapd Bennett Todd (Jul 16)
- Re: FWTK and smap/smapd Russell Van Tassell (Jul 16)
- Re: FWTK and smap/smapd Joseph S D Yao (Jul 17)
- Re: FWTK and smap/smapd Frederick M Avolio (Jul 17)
- Re: FWTK and smap/smapd Joseph S D Yao (Jul 17)
- Re: FWTK and smap/smapd Marcus J. Ranum (Jul 19)
- Re: FWTK and smap/smapd Joseph S D Yao (Jul 19)
- Re: FWTK and smap/smapd Marcus J. Ranum (Jul 19)
- Re: FWTK and smap/smapd Anton J Aylward, CISSP (Jul 19)
- Re: FWTK and smap/smapd Paul D. Robertson (Jul 19)
- Re: FWTK and smap/smapd R. DuFresne (Jul 19)
- Re: FWTK and smap/smapd Joseph S D Yao (Jul 19)
- Message not available
- Code reviews [Was: FWTK and smap/smapd] Marcus J. Ranum (Jul 19)
- Re: Code reviews [Was: FWTK and smap/smapd] Jim Duncan (Jul 19)
- Re: Code reviews [Was: FWTK and smap/smapd] Carson Gaspar (Jul 22)
- Re: Code reviews [Was: FWTK and smap/smapd] ark (Jul 23)
- Re: FWTK and smap/smapd Russell Van Tassell (Jul 16)