Re: man page nft.8 add chain synopsis

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Duncan,

On Fri, Oct 06, 2017 at 04:08:38PM +1100, Duncan Roe wrote:
> Hi,
> 
> The man page says this:
> 
> > {add | create} chain [family] table chain [ { {type} {hook} [device] {priority} } [policy] ]
> 
> But I suggest it should say this:
> 
> > {add | create} chain [family] table chain [ { {type} {hook} [device] {priority ;} [policy ;] } ]
> 
> i.e. the policy tuple should be inside the braces along with type, hook, device
> & priority. Also the device & priority tuples each need to be followed by a
> semicolon.
> 
> ** Is the above change correct?

This looks good to me.

I will include your Signed-off-by tag and will take this into the git
tree.

BTW, please next time submit your patches via git format-patch.

> * Assuming it is correct, the patch below makes the change. Can someone pick it
>   up from here or should I submit a bug report?

I'll pick it, no need for bug report.

Thanks!
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux