]> git.ipfire.org Git - thirdparty/iptables.git/commitdiff
Lift restrictions on interface names
authorJan Engelhardt <jengelh@medozas.de>
Tue, 9 Feb 2010 14:59:13 +0000 (15:59 +0100)
committerPatrick McHardy <kaber@trash.net>
Tue, 9 Feb 2010 14:59:13 +0000 (15:59 +0100)
The kernel has few restrictions.

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
Signed-off-by: Patrick McHardy <kaber@trash.net>
xtables.c

index 63c5db77270f28b123fd04acf5f79628de1e8d5e..f3baf84d2fd557ce63914bea823c04e1d0db92d5 100644 (file)
--- a/xtables.c
+++ b/xtables.c
@@ -450,7 +450,7 @@ u_int16_t xtables_parse_port(const char *port, const char *proto)
 void xtables_parse_interface(const char *arg, char *vianame,
                             unsigned char *mask)
 {
-       int vialen = strlen(arg);
+       unsigned int vialen = strlen(arg);
        unsigned int i;
 
        memset(mask, 0, IFNAMSIZ);
@@ -462,7 +462,7 @@ void xtables_parse_interface(const char *arg, char *vianame,
                           " (%i)", arg, IFNAMSIZ-1);
 
        strcpy(vianame, arg);
-       if ((vialen == 0) || (vialen == 1 && vianame[0] == '+'))
+       if (vialen == 0)
                memset(mask, 0, IFNAMSIZ);
        else if (vianame[vialen - 1] == '+') {
                memset(mask, 0xFF, vialen - 1);
@@ -473,12 +473,11 @@ void xtables_parse_interface(const char *arg, char *vianame,
                memset(mask, 0xFF, vialen + 1);
                memset(mask + vialen + 1, 0, IFNAMSIZ - vialen - 1);
                for (i = 0; vianame[i]; i++) {
-                       if (vianame[i] == ':' ||
-                           vianame[i] == '!' ||
-                           vianame[i] == '*') {
+                       if (vianame[i] == '/' ||
+                           vianame[i] == ' ') {
                                fprintf(stderr,
                                        "Warning: weird character in interface"
-                                       " `%s' (No aliases, :, ! or *).\n",
+                                       " `%s' ('/' and ' ' are not allowed by the kernel).\n",
                                        vianame);
                                break;
                        }