Re: iptables manpage and ICMP type/code

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

 



On Tuesday 2009-01-13 15:08, Victor Stinner wrote:
>iptables supports ICMP code filtering, but it's not documented in the manpage.
>Attached patch fixes the manpage, [...]:
>-       --icmp-type [!] typename
>+       --icmp-type [!] type[/code]|typename
>              This allows specification of the ICMP type, which can be a
>              the command
>               iptables -p icmp -h
>
>It changes also the text style: keywords are underlined.

Actually \fI is italic (it's "I" for a reason); it's just that,
now that you mention it, there seems to be a glitch in `man`
that causes it to be displayed as underline.

>-[\fB!\fP] \fB\-\-icmp\-type\fP \fItypename\fP
>+\fB\-\-icmp\-type\fP [\fB!\fP] \fItype\fP[\fB/\fP\fIcode\fP]|\fItypename\fP

This is old syntax.

> This allows specification of the ICMP type, which can be a numeric
>-ICMP type, or one of the ICMP type names shown by the command
>+ICMP
>+.IR type ,
>+.IR type
>+and
>+.IR code ,

'type' did not refer to the placeholder, but the type in general,
hence it was not italic to begin with. I think this should not
change, either.



parent 3e95daeb83446a61afa86914c9f1a03fa257537d (v1.4.3-rc1-19-g3e95dae)
commit aa171d33c6d86940a057db2bd037f2f0106f1221
Author: Jan Engelhardt <jengelh@xxxxxxxxxx>
Date:   Tue Jan 13 15:30:10 2009 +0100

doc: augment ICMP manpage by type/code syntax

The ICMP match module also allows numeric type/code as in
"--icmp-type type/code". Document it.

Based upon a patch by Victor Stinner <vstinner@xxxxxx>.

Signed-off-by: Jan Engelhardt <jengelh@xxxxxxxxxx>
---
 extensions/libipt_icmp.c   |    2 +-
 extensions/libipt_icmp.man |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/extensions/libipt_icmp.c b/extensions/libipt_icmp.c
index d0b7bb3..fa5a540 100644
--- a/extensions/libipt_icmp.c
+++ b/extensions/libipt_icmp.c
@@ -102,7 +102,7 @@ static void icmp_help(void)
 	printf(
 "icmp match options:\n"
 "[!] --icmp-type typename	match icmp type\n"
-"				(or numeric type or type/code)\n");
+"[!] --icmp-type type[/code]	(or numeric type or type/code)\n");
 	print_icmptypes();
 }
 
diff --git a/extensions/libipt_icmp.man b/extensions/libipt_icmp.man
index 8c1bdbe..1039704 100644
--- a/extensions/libipt_icmp.man
+++ b/extensions/libipt_icmp.man
@@ -1,9 +1,9 @@
 This extension can be used if `\-\-protocol icmp' is specified. It
 provides the following option:
 .TP
-[\fB!\fP] \fB\-\-icmp\-type\fP \fItypename\fP
+[\fB!\fP] \fB\-\-icmp\-type\fP {\fItype\fP[\fB/\fP\fIcode\fP]|\fItypename\fP}
 This allows specification of the ICMP type, which can be a numeric
-ICMP type, or one of the ICMP type names shown by the command
+ICMP type, type/code pair, or one of the ICMP type names shown by the command
 .nf
  iptables \-p icmp \-h
 .fi
-- 
# Created with git-export-patch
--
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