Package:  iptables
Version:  1.4.9-1
Severity: wishlist
Tags:     patch

  I think the addition of numeric types/codes to the output of 
    iptables -p icmp -h
Would make it clearer, in particular when referring to other texts.

--- extensions/libipt_icmp.c.orig       2010-10-21 04:43:27.000000000 +0300
+++ extensions/libipt_icmp.c    2010-10-21 14:31:15.000000000 +0300
@@ -81,7 +81,7 @@
 print_icmptypes(void)
 {
        unsigned int i;
-       printf("Valid ICMP Types:");
+       printf("Valid, numeric values and named, ICMP Types and codes:");

        for (i = 0; i < ARRAY_SIZE(icmp_codes); ++i) {
                if (i && icmp_codes[i].type == icmp_codes[i-1].type) {
@@ -90,10 +90,12 @@
                                == icmp_codes[i-1].code_max))
                                printf(" (%s)", icmp_codes[i].name);
                        else
-                               printf("\n   %s", icmp_codes[i].name);
+                               printf("\n%5s%2u%2s%s", "", 
icmp_codes[i].code_min,
+                                                       "", icmp_codes[i].name);
                }
                else
-                       printf("\n%s", icmp_codes[i].name);
+                       printf("\n%2s%3u%2s%s", "", icmp_codes[i].type,
+                                               "", icmp_codes[i].name);
        }
        printf("\n");
 }
 
                                          


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to