Numeric interface names

[profplump]

radvd will not accept numeric-only interface names, producing a "syntax error" in config parsing

This doesn't work:

interface 4114
{
	...
};

But this does:

interface f114
{

I tried double-quotes to delimit, and while the quoted version parses without error radvd then expects literal double-quotes in the interface name, which of course doesn't exist

interface "4114"
{

Is there another delimiter/syntax I should be using? If not, is this a limitation in radvd config parser, or are numeric-only interface names intentionally not supported?

[reubenhwk]

The scanner is regex matching number before string. There is no rule in the grammar for "interface" token followed by "number" token... Just a rule for "interface" followed by "string". I suppose it would not be too difficult to add a rule for that though..

…

On Fri, Aug 8, 2025 at 5:53 PM Zach Isbach ***@***.***> wrote: *profplump* created an issue (radvd-project/radvd#272) <#272> radvd will not accept numeric-only interface names, producing a "syntax error" in config parsing This doesn't work: interface 4114 { ... }; But this does: interface f114 { I tried double-quotes to delimit, and while the quoted version parses without error radvd then expects literal double-quotes in the interface name, which of course doesn't exist interface "4114" { Is there another delimiter/syntax I should be using? If not, is this a limitation in radvd config parser, or are numeric-only interface names intentionally not supported? — Reply to this email directly, view it on GitHub <#272>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABRG63AFP3FBGX5SLRBBJT3MUS7ZAVCNFSM6AAAAACDPBRG6SVHI2DSMVQWIX3LMV43ASLTON2WKOZTGMYDKMZZGQ2TANA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[reubenhwk]

Not too difficult to add NUMBER to the grammar like this... $ git diff diff --git a/gram.y b/gram.y index 45b2ff7..9beb96a 100644 --- a/gram.y +++ b/gram.y @@ -232,6 +232,15 @@ name : STRING /* check vality */ $$ = $1; } + | NUMBER + { + static char string[256]; + + snprintf(string, sizeof(string), "%d", $1); + yylval.str = string; + + $$ = yylval.str; + } ; Example output... $ ./radvd -C test.conf -d 5 -m stderr [Aug 09 00:39:35] radvd (289213): version 2.20 started [Aug 09 00:39:35] radvd (289213): 4114 interface definition ok [Aug 09 00:39:35] radvd (289213): eth0 interface definition ok I'm wondering about also adding DECIMAL to the grammar so the next guy will also be able to do this... interface 4114.1 { };

…

On Sat, Aug 9, 2025 at 12:29 AM Reuben Hawkins ***@***.***> wrote: The scanner is regex matching number before string. There is no rule in the grammar for "interface" token followed by "number" token... Just a rule for "interface" followed by "string". I suppose it would not be too difficult to add a rule for that though.. On Fri, Aug 8, 2025 at 5:53 PM Zach Isbach ***@***.***> wrote: > *profplump* created an issue (radvd-project/radvd#272) > <#272> > > radvd will not accept numeric-only interface names, producing a "syntax > error" in config parsing > > This doesn't work: > > interface 4114 > { > ... > }; > > But this does: > > interface f114 > { > > I tried double-quotes to delimit, and while the quoted version parses > without error radvd then expects literal double-quotes in the interface > name, which of course doesn't exist > > interface "4114" > { > > Is there another delimiter/syntax I should be using? If not, is this a > limitation in radvd config parser, or are numeric-only interface names > intentionally not supported? > > — > Reply to this email directly, view it on GitHub > <#272>, or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AABRG63AFP3FBGX5SLRBBJT3MUS7ZAVCNFSM6AAAAACDPBRG6SVHI2DSMVQWIX3LMV43ASLTON2WKOZTGMYDKMZZGQ2TANA> > . > You are receiving this because you are subscribed to this thread.Message > ID: ***@***.***> >

[reubenhwk]

Ok I have code changes that can support interface names such as '4114' and '4114.1' I've barely tested it. I'll push it up for review at some point. There are some static strings in there that make me nervous. It's been a while since I've worked on this code.

…

On Sat, Aug 9, 2025 at 12:45 AM Reuben Hawkins ***@***.***> wrote: Not too difficult to add NUMBER to the grammar like this... $ git diff diff --git a/gram.y b/gram.y index 45b2ff7..9beb96a 100644 --- a/gram.y +++ b/gram.y @@ -232,6 +232,15 @@ name : STRING /* check vality */ $$ = $1; } + | NUMBER + { + static char string[256]; + + snprintf(string, sizeof(string), "%d", $1); + yylval.str = string; + + $$ = yylval.str; + } ; Example output... $ ./radvd -C test.conf -d 5 -m stderr [Aug 09 00:39:35] radvd (289213): version 2.20 started [Aug 09 00:39:35] radvd (289213): 4114 interface definition ok [Aug 09 00:39:35] radvd (289213): eth0 interface definition ok I'm wondering about also adding DECIMAL to the grammar so the next guy will also be able to do this... interface 4114.1 { }; On Sat, Aug 9, 2025 at 12:29 AM Reuben Hawkins ***@***.***> wrote: > The scanner is regex matching number before string. There is no rule in > the grammar for "interface" token followed by "number" token... Just a > rule for "interface" followed by "string". I suppose it would not be too > difficult to add a rule for that though.. > > On Fri, Aug 8, 2025 at 5:53 PM Zach Isbach ***@***.***> > wrote: > >> *profplump* created an issue (radvd-project/radvd#272) >> <#272> >> >> radvd will not accept numeric-only interface names, producing a "syntax >> error" in config parsing >> >> This doesn't work: >> >> interface 4114 >> { >> ... >> }; >> >> But this does: >> >> interface f114 >> { >> >> I tried double-quotes to delimit, and while the quoted version parses >> without error radvd then expects literal double-quotes in the interface >> name, which of course doesn't exist >> >> interface "4114" >> { >> >> Is there another delimiter/syntax I should be using? If not, is this a >> limitation in radvd config parser, or are numeric-only interface names >> intentionally not supported? >> >> — >> Reply to this email directly, view it on GitHub >> <#272>, or unsubscribe >> <https://github.com/notifications/unsubscribe-auth/AABRG63AFP3FBGX5SLRBBJT3MUS7ZAVCNFSM6AAAAACDPBRG6SVHI2DSMVQWIX3LMV43ASLTON2WKOZTGMYDKMZZGQ2TANA> >> . >> You are receiving this because you are subscribed to this thread.Message >> ID: ***@***.***> >> >

[reubenhwk]

If I forget to push the branch, here is the patch.

…

On Sat, Aug 9, 2025 at 1:14 AM Reuben Hawkins ***@***.***> wrote: Ok I have code changes that can support interface names such as '4114' and '4114.1' I've barely tested it. I'll push it up for review at some point. There are some static strings in there that make me nervous. It's been a while since I've worked on this code. On Sat, Aug 9, 2025 at 12:45 AM Reuben Hawkins ***@***.***> wrote: > Not too difficult to add NUMBER to the grammar like this... > > $ git diff > diff --git a/gram.y b/gram.y > index 45b2ff7..9beb96a 100644 > --- a/gram.y > +++ b/gram.y > @@ -232,6 +232,15 @@ name : STRING > /* check vality */ > $$ = $1; > } > + | NUMBER > + { > + static char string[256]; > + > + snprintf(string, sizeof(string), "%d", $1); > + yylval.str = string; > + > + $$ = yylval.str; > + } > ; > > Example output... > > $ ./radvd -C test.conf -d 5 -m stderr > [Aug 09 00:39:35] radvd (289213): version 2.20 started > [Aug 09 00:39:35] radvd (289213): 4114 interface definition ok > [Aug 09 00:39:35] radvd (289213): eth0 interface definition ok > > I'm wondering about also adding DECIMAL to the grammar so the next guy > will also be able to do this... > > interface 4114.1 { > }; > > On Sat, Aug 9, 2025 at 12:29 AM Reuben Hawkins ***@***.***> > wrote: > >> The scanner is regex matching number before string. There is no rule in >> the grammar for "interface" token followed by "number" token... Just a >> rule for "interface" followed by "string". I suppose it would not be too >> difficult to add a rule for that though.. >> >> On Fri, Aug 8, 2025 at 5:53 PM Zach Isbach ***@***.***> >> wrote: >> >>> *profplump* created an issue (radvd-project/radvd#272) >>> <#272> >>> >>> radvd will not accept numeric-only interface names, producing a "syntax >>> error" in config parsing >>> >>> This doesn't work: >>> >>> interface 4114 >>> { >>> ... >>> }; >>> >>> But this does: >>> >>> interface f114 >>> { >>> >>> I tried double-quotes to delimit, and while the quoted version parses >>> without error radvd then expects literal double-quotes in the interface >>> name, which of course doesn't exist >>> >>> interface "4114" >>> { >>> >>> Is there another delimiter/syntax I should be using? If not, is this a >>> limitation in radvd config parser, or are numeric-only interface names >>> intentionally not supported? >>> >>> — >>> Reply to this email directly, view it on GitHub >>> <#272>, or unsubscribe >>> <https://github.com/notifications/unsubscribe-auth/AABRG63AFP3FBGX5SLRBBJT3MUS7ZAVCNFSM6AAAAACDPBRG6SVHI2DSMVQWIX3LMV43ASLTON2WKOZTGMYDKMZZGQ2TANA> >>> . >>> You are receiving this because you are subscribed to this thread.Message >>> ID: ***@***.***> >>> >>