[radvd-devel-l] Radvd fails to handle interface names with capital letters in them.

Pekka Savola pekkas at netcore.fi
Tue Jun 24 16:30:52 EDT 2008


Hi Ben,

An interesting perspective indeed.  We've just been happy with the 
path of least resistance and always patched the definition.  Off the 
top of my head, I guess the restrictions could be removed.  Anyone 
else have an opinion?  Someone want to cook up a patch to do this?

On Thu, 19 Jun 2008, Ben Greear wrote:
> This patch appears to fix the problem, though I haven't
> actually tested full functionality.  That said, why is
> it so restrictive to begin with?  Why not just the equivalent
> of \S+ in perl-speak?  Mac-vlans have a # in their name by
> default, for example, and users can rename it to anything they
> want anyway...
>
> diff -u radvd-1.1/scanner.l  radvd-1.1.ben/scanner.l
> --- radvd-1.1/scanner.l 2007-10-25 13:34:55.000000000 -0700
> +++ radvd-1.1.ben/scanner.l     2008-06-19 12:08:41.000000000 -0700
> @@ -34,7 +34,7 @@
> addr3          ({hexdigit}{1,4}":"){7}{hexdigit}{1,4}
> addr           ({addr1}|{addr2}|{addr3}|"::")
> whitespace     ([ \t])+
> -string         [a-z]([a-z:._-]|{digit})*([:.]{digit}+)?
> +string         [a-zA-Z]([a-zA-Z:._-]|{digit})*([:.]{digit}+)?
> %%
>
> #.*$                   {/* ignore comments */}
>
>
>
>

-- 
Pekka Savola                 "You each name yourselves king, yet the
Netcore Oy                    kingdom bleeds."
Systems. Networks. Security. -- George R.R. Martin: A Clash of Kings



More information about the radvd-devel-l mailing list