[U-Boot] [PATCH] net: fix env flags for eth10addr and above
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
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?
+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?
Regards, Simon
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.
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.
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...
Regards, Simon
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.
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.
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
Hi Joe,
Am 14.01.2019 um 19:19 schrieb Joe Hershberger:
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.
I thought of something like that, given the time in the year :-)
Note that I sent 4 patches for net that are presumably in your area to pick up. This one and:
https://patchwork.ozlabs.org/patch/1001862/ https://patchwork.ozlabs.org/patch/999267/ https://patchwork.ozlabs.org/patch/999268/
Thanks, Simon
-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
On Thu, Nov 22, 2018 at 10:06 AM Simon Goldschmidt simon.k.r.goldschmidt@gmail.com wrote:
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
Acked-by: Joe Hershberger joe.hershberger@ni.com
Hi Simon,
https://patchwork.ozlabs.org/patch/1001866/ was applied to http://git.denx.de/?p=u-boot/u-boot-net.git
Thanks! -Joe
participants (3)
-
Joe Hershberger -
Simon Goldschmidt -
Tom Rini