Hi Simon,
On Mon, Jan 14, 2019 at 10:29 AM Tom Rini trini@konsulko.com wrote:
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.goldschmidt@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.goldschmidt@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.
Apologies for the dead air... I was out for about a month and then the holidays hit. I've been trying to catch up on things that languished in that time and missed this one.
-Joe
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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot