Subject: Re: CVS commit: src/etc
To: Martin Husemann <martin@NetBSD.org>
From: Christos Zoulas <christos@zoulas.com>
List: tech-net
Date: 05/17/2007 10:05:42
On May 17, 11:26am, martin@NetBSD.org (Martin Husemann) wrote:
-- Subject: Re: CVS commit: src/etc

| On Tue, May 08, 2007 at 08:40:12PM +0000, Christos Zoulas wrote:
| > 
| > Module Name:	src
| > Committed By:	christos
| > Date:		Tue May  8 20:40:12 UTC 2007
| > 
| > Modified Files:
| > 	src/etc: protocols
| > 
| > Log Message:
| > iana-generated protocols file using:
| >     http://www.sethwklein.net/projects/iana-etc/
| 
| This breaks (at least) ipfilter, by defining protocol 0 to be hopopt.
| Seems so far only some AIX variant did that, so I'd expect more lossage
| in other applications. (But then, not a lot will care, probably)

So does linux, but ipf does not run on it.

| For fun, go to src/dist/ipf and fgrep -r for _AIX51 and check those ifdefs.
| We could just add || defined(__NetBSD__) there, but I'm not sure that is
| the right fix.

IANA says both ip and hopopt are protocol 0. I think that ipf should
be better off leaving the numbers untranslated in the unit-tests.
Why translate ports and services to strings? Yes, doing || defined(__NetBSD__)
would be better.

christos