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.

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

ugh!

jmc

Reply via email to