* Jason McIntyre <[email protected]> [2009-05-30 11:16]: > On Sat, May 30, 2009 at 09:16:56AM +0100, Federico G. Schwindt wrote: > > > > > > 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) > > so the document is currently incorrect? we need to document this?
well the BNF is correct. we might want to add a note to the return-rst description tho. > > > 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. > yes, but can it take an argument of an icmp*6* code? i know return-icmp > can take an icmp code, and return-icmp6 can take an icmp6 code, but > return-icmp can take a 6 code (or vice versa)? surely that is wrong. return-icmp works for both address families. -- Henning Brauer, [email protected], [email protected] BS Web Services, http://bsws.de Full-Service ISP - Secure Hosting, Mail and DNS Services Dedicated Servers, Rootservers, Application Hosting - Hamburg & Amsterdam
