On Sat, May 30, 2009 at 07:49:10AM +0100, Jason McIntyre wrote:
> On Fri, May 29, 2009 at 11:59:54PM +0100, Federico G. Schwindt wrote:
> > On Sat, May 30, 2009 at 12:17:16AM +0200, Laurent Ghigonis wrote:
> > > Hi here :)
> > > 
> > > i guess i found a typo in the pf.conf man page, syntax section.
> > > 
> > > anyway, keep up the great work devs !
> > > clean and simple ... love it
> > 
> >   that's actually correct. ttl is part of the string.
> > 
> >   f.-
> 
> is it correct? this whole bit looks a bit bizarre. let me quote it
> again:
> 
>     return = "drop" | "return" | "return-rst" [ "( ttl" number ")" ] |
>              "return-icmp" [ "(" icmpcode [ [ "," ] icmp6code ] ")" ] |
>            "return-icmp6" [ "(" icmp6code ")" ]
> 
> first off, return-rst makes no note in the doc that it can take "ttl <n>"
> as an argument. can it really do that? even if it can, the quoting still
> looks incorrect.

  yes, it's like that.  how the quote will be in the case of:

  return-rst(ttl 20)

> second, can return-icmp really take an icmp6code as an argument? is that
> what the syntax is saying?

  yes, it can take a second argument.

  f.-

Reply via email to