From: Adrian Freihofer <[email protected]>

In explicit write access mode (CONFIG_ENV_WRITEABLE_LIST) the
environment variables listed in NET_FLAGS and NET6_FLAGS are
probably intended to be writeable. Therefore add the 'w' flag
to these variables.

Signed-off-by: Adrian Freihofer <[email protected]>
Signed-off-by: Heiko Schocher <[email protected]>
---

 include/env_flags.h | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

Azure build (patch is in a bunch of siemens patches sending
now to mainline):
https://dev.azure.com/hs0298/hs/_build/results?buildId=202&view=results

diff --git a/include/env_flags.h b/include/env_flags.h
index 0c48874690f..fc65dcaba21 100644
--- a/include/env_flags.h
+++ b/include/env_flags.h
@@ -56,6 +56,16 @@ enum env_flags_varaccess {
 #define ETHADDR_FLAGS "eth" ETHADDR_WILDCARD "addr:mo,"
 #endif
 #endif
+#ifdef CONFIG_ENV_WRITEABLE_LIST
+#define NET_FLAGS \
+       "ipaddr:iw," \
+       "gatewayip:iw," \
+       "netmask:iw," \
+       "serverip:iw," \
+       "nvlan:dw," \
+       "vlan:dw," \
+       "dnsip:iw,"
+#else
 #define NET_FLAGS \
        "ipaddr:i," \
        "gatewayip:i," \
@@ -64,16 +74,24 @@ enum env_flags_varaccess {
        "nvlan:d," \
        "vlan:d," \
        "dnsip:i,"
+#endif
 #else
 #define ETHADDR_FLAGS
 #define NET_FLAGS
 #endif
 
 #ifdef CONFIG_IPV6
+#ifdef CONFIG_ENV_WRITEABLE_LIST
+#define NET6_FLAGS \
+       "ip6addr:sw," \
+       "serverip6:sw," \
+       "gatewayip6:sw,"
+#else
 #define NET6_FLAGS \
        "ip6addr:s," \
        "serverip6:s," \
        "gatewayip6:s,"
+#endif
 #else
 #define NET6_FLAGS
 #endif
-- 
2.20.1

base-commit: 6b2d05748cf3cd6ba417a96c00602b0122e10af6

Reply via email to