Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 23 Oct 2001 19:45:11 +0200
From:      Gerhard Sittig <Gerhard.Sittig@gmx.net>
To:        Darren Reed <darrenr@FreeBSD.ORG>
Cc:        cvs-all@FreeBSD.ORG
Subject:   Re: cvs commit: src/etc rc.network
Message-ID:  <20011023194511.D97313@shell.gsinet.sittig.org>
In-Reply-To: <20011023145506.C20345@hades.hell.gr>; from charon@labs.gr on Tue, Oct 23, 2001 at 02:55:06PM %2B0300
References:  <20011023012512.A1596@hades.hell.gr> <58940.1003834403@axl.seasidesoftware.co.za> <20011023145506.C20345@hades.hell.gr>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Oct 23, 2001 at 14:55 +0300, Giorgos Keramidas wrote:
> On Tue, Oct 23, 2001 at 12:53:23PM +0200, Sheldon Hearn wrote:
> > On Tue, 23 Oct 2001 01:25:12 +0300, Giorgos Keramidas wrote:
> > > With ipfilter_program set to "/sbin/ipf -Fa -f" (as rc.conf(5) says)
> > > in /etc/defaults/rc.conf, this change breaks rc.network in the part
> > > that calls `ipf -y'.  The code calls:
> >
> > foo_program should never include flags.
> 
> Yes, that's what I think too, but I'd hate to step on Darren's toes,
> if he has these set like that for a purpose.

I get the feeling this - inappropriate - setting of a _program
variable is due to my misguided suggestion in PR conf/20202
which verbatimly made it into the FreeBSD start scripts.  If it
doesn't fit the usual rules feel free to correct it! :)  After
all I was a newbee to FreeBSD then (and still I'm not a guru or
seasoned hacker:) as well as I understand Darren to do his
daytime job with SunOS / Solaris and since he might need some
hints on how his software fits even better into FreeBSD.  I guess
he will happily accept patches improving a wrong approach.

Maybe there's need for the following parts:
- ipfilter_program
- ipfilter_prerules_flags
- ipfilter_rules
- ipfilter_postrules_flags
?  The current situation comes from the fact that I wanted to
have a single variable with the rules file only - to check for
its existance (if such an additional constraints check matters).

> After I posted my diff,
> I saw Arjan de Vet had posted a patch that makes ipfilter_program work
> with a value of "/sbin/ipf".  Let's go for Arjan's changes instead.

If there's already a correcting / much improving patch maybe it
should be in the PR database?  Darren Reed apparently works on
FreeBSD tasks in bursts and might appreciate a PR with its better
record / specific assignment more than a rushing by article
buried in the high volume of a mailing list ...


virtually yours   82D1 9B9C 01DC 4FB4 D7B4  61BE 3F49 4F77 72DE DA76
Gerhard Sittig   true | mail -s "get gpg key" Gerhard.Sittig@gmx.net
-- 
     If you don't understand or are scared by any of the above
             ask your parents or an adult to help you.

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20011023194511.D97313>