On Mon, Jan 14, 2019 at 04:42:10PM +0100, Simon Goldschmidt wrote:
> Am 14.12.2018 um 14:28 schrieb Tom Rini:
> >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.
> 
> Will you be taking this after the release or is there anything missing? It's
> assigned to you in patchwork but I didn't get any real reply...

Yes, I'll be picking it up then, thanks for your patience.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to