[U-Boot] [PATCH] net: fix env flags for eth10addr and above

Simon Goldschmidt simon.k.r.goldschmidt at gmail.com
Mon Jan 14 15:42:10 UTC 2019


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 at 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 at 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


More information about the U-Boot mailing list