All of lore.kernel.org
 help / color / mirror / Atom feed
* conntrackd refuses to work on interface with '_' in it's name
@ 2015-02-17 10:36 AZ
  2015-02-17 22:11 ` Paul Robert Marino
  2015-02-18 23:58 ` Pablo Neira Ayuso
  0 siblings, 2 replies; 3+ messages in thread
From: AZ @ 2015-02-17 10:36 UTC (permalink / raw)
  To: netfilter-devel

There is a syntax error if you'll try to use interface with the name
containing underscore.
The kernel works with such an interfaces, so there's no reason
conntrackd shouldn't.
I propose the following patch to fix this, works for me:
--- read_config_lex.l.orig      2015-02-17 16:35:08.160554333 +0600
+++ read_config_lex.l   2015-02-17 16:35:24.053887237 +0600
@@ -47,7 +47,7 @@ ip6_part      {hex_255}":"?
 ip6_form1      {ip6_part}{0,7}"::"{ip6_part}{0,7}
 ip6_form2      ({hex_255}":"){0,7}{hex_255}
 ip6            {ip6_form1}{ip6_cidr}?|{ip6_form2}{ip6_cidr}?
-string         [a-zA-Z][a-zA-Z0-9\.\-]*
+string         [a-zA-Z][a-zA-Z0-9_\.\-]*
 persistent     [P|p][E|e][R|r][S|s][I|i][S|s][T|t][E|e][N|n][T|T]
 nack           [N|n][A|a][C|c][K|k]
 alarm          [A|a][L|l][A|a][R|r][M|m]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: conntrackd refuses to work on interface with '_' in it's name
  2015-02-17 10:36 conntrackd refuses to work on interface with '_' in it's name AZ
@ 2015-02-17 22:11 ` Paul Robert Marino
  2015-02-18 23:58 ` Pablo Neira Ayuso
  1 sibling, 0 replies; 3+ messages in thread
From: Paul Robert Marino @ 2015-02-17 22:11 UTC (permalink / raw)
  To: AZ; +Cc: netfilter-devel

The iproute2 commands really don't put many restrictions on interface
names but most people generally avoid strange names because of
compatibility issues, and the fact that it can be confusing for admins
who are not properly taught the logic behind the custom names.
While the idea is not bad per say it is definitely an edge case.

On Tue, Feb 17, 2015 at 5:36 AM, AZ <ak47izatool@gmail.com> wrote:
> There is a syntax error if you'll try to use interface with the name
> containing underscore.
> The kernel works with such an interfaces, so there's no reason
> conntrackd shouldn't.
> I propose the following patch to fix this, works for me:
> --- read_config_lex.l.orig      2015-02-17 16:35:08.160554333 +0600
> +++ read_config_lex.l   2015-02-17 16:35:24.053887237 +0600
> @@ -47,7 +47,7 @@ ip6_part      {hex_255}":"?
>  ip6_form1      {ip6_part}{0,7}"::"{ip6_part}{0,7}
>  ip6_form2      ({hex_255}":"){0,7}{hex_255}
>  ip6            {ip6_form1}{ip6_cidr}?|{ip6_form2}{ip6_cidr}?
> -string         [a-zA-Z][a-zA-Z0-9\.\-]*
> +string         [a-zA-Z][a-zA-Z0-9_\.\-]*
>  persistent     [P|p][E|e][R|r][S|s][I|i][S|s][T|t][E|e][N|n][T|T]
>  nack           [N|n][A|a][C|c][K|k]
>  alarm          [A|a][L|l][A|a][R|r][M|m]
> --
> To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: conntrackd refuses to work on interface with '_' in it's name
  2015-02-17 10:36 conntrackd refuses to work on interface with '_' in it's name AZ
  2015-02-17 22:11 ` Paul Robert Marino
@ 2015-02-18 23:58 ` Pablo Neira Ayuso
  1 sibling, 0 replies; 3+ messages in thread
From: Pablo Neira Ayuso @ 2015-02-18 23:58 UTC (permalink / raw)
  To: AZ; +Cc: netfilter-devel

On Tue, Feb 17, 2015 at 05:36:10PM +0700, AZ wrote:
> There is a syntax error if you'll try to use interface with the name
> containing underscore.
> The kernel works with such an interfaces, so there's no reason
> conntrackd shouldn't.

Just applied a similar patch, this one was not applying cleanly with
via git am.

> I propose the following patch to fix this, works for me:
> --- read_config_lex.l.orig      2015-02-17 16:35:08.160554333 +0600
> +++ read_config_lex.l   2015-02-17 16:35:24.053887237 +0600
> @@ -47,7 +47,7 @@ ip6_part      {hex_255}":"?
>  ip6_form1      {ip6_part}{0,7}"::"{ip6_part}{0,7}
>  ip6_form2      ({hex_255}":"){0,7}{hex_255}
>  ip6            {ip6_form1}{ip6_cidr}?|{ip6_form2}{ip6_cidr}?
> -string         [a-zA-Z][a-zA-Z0-9\.\-]*
> +string         [a-zA-Z][a-zA-Z0-9_\.\-]*
>  persistent     [P|p][E|e][R|r][S|s][I|i][S|s][T|t][E|e][N|n][T|T]
>  nack           [N|n][A|a][C|c][K|k]
>  alarm          [A|a][L|l][A|a][R|r][M|m]
> --
> To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-02-18 23:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-17 10:36 conntrackd refuses to work on interface with '_' in it's name AZ
2015-02-17 22:11 ` Paul Robert Marino
2015-02-18 23:58 ` Pablo Neira Ayuso

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.