On Fri, Dec 14, 2018 at 09:56:19AM +0100, Simon Goldschmidt wrote:
> +Tom
>
> On Sun, Dec 9, 2018 at 9:51 PM Simon Goldschmidt
> <simon.k.r.goldschm...@gmail.com> wrote:
> >
> > Am 22.11.2018 um 17:06 schrieb Simon Goldschmidt:
> > > With CONFIG_REGEX enabled, ETHADDR_WILDCARD is set up for up to 10
> > > interfaces (0..9) as the number can only have one digit.
> > >
> > > On boards with more than 10 interfaces, this leads to the protection
> > > and format checks being absent for eth10addr and above.
> > >
> > > Fix this by changing ETHADDR_WILDCARD from "\\d?" to "\\d*" to allow
> > > more than one digit.
> > >
> > > Signed-off-by: Simon Goldschmidt <simon.k.r.goldschm...@gmail.com>
> > > ---
> > >
> > > include/env_flags.h | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/include/env_flags.h b/include/env_flags.h
> > > index cc2c34f939..23744e395c 100644
> > > --- a/include/env_flags.h
> > > +++ b/include/env_flags.h
> > > @@ -38,7 +38,7 @@ enum env_flags_varaccess {
> > >
> > > #ifdef CONFIG_CMD_NET
> > > #ifdef CONFIG_REGEX
> > > -#define ETHADDR_WILDCARD "\\d?"
> > > +#define ETHADDR_WILDCARD "\\d*"
> > > #else
> > > #define ETHADDR_WILDCARD
> > > #endif
> > >
> >
> > Ping?
>
> I haven't gotten an answer to this for nearly one month. It's a small
> change, can this go in for v2019.01 please?I'd filed this off to Joe to grab since it is indeed small and a clean-up. -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
