Discussion:
CVS commit: src/etc
(too old to reply)
Martin Husemann
2007-05-17 11:26:52 UTC
Permalink
Module Name: src
Committed By: christos
Date: Tue May 8 20:40:12 UTC 2007
src/etc: protocols
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)

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.

Thoughts?

Martin

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de
Christos Zoulas
2007-05-17 14:05:42 UTC
Permalink
On May 17, 11:26am, ***@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

--
Posted automagically by a mail2news gateway at muc.de e.V.
Please direct questions, flames, donations, etc. to news-***@muc.de
Loading...