diff options
| author | Sven Eckelmann | 2025-10-05 07:00:08 +0000 |
|---|---|---|
| committer | Hauke Mehrtens | 2025-10-12 13:52:12 +0000 |
| commit | 18e1929401bed72a6e4f7b9b4ccee81cc67a47d6 (patch) | |
| tree | e82ace0ae80369a590cd8783f4f69a5137b20291 | |
| parent | f0648fd5760c3f9ec036d1a51a41c8eedd399ec6 (diff) | |
| download | openwrt-18e1929401bed72a6e4f7b9b4ccee81cc67a47d6.tar.gz | |
realtek: Avoid empty lan mac in initial network setup
If the lan_mac cannot be found, it is still used (as empty string) in
various operations. This is not valid and other 02_network scripts checking
for a non-empty string before using it. This should also be adopted for the
realtek 02_network.
Signed-off-by: Sven Eckelmann <se@simonwunderlich.de>
Link: https://github.com/openwrt/openwrt/pull/20241
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
| -rw-r--r-- | target/linux/realtek/base-files/etc/board.d/02_network | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/target/linux/realtek/base-files/etc/board.d/02_network b/target/linux/realtek/base-files/etc/board.d/02_network index 79c140c32a..464ff976cf 100644 --- a/target/linux/realtek/base-files/etc/board.d/02_network +++ b/target/linux/realtek/base-files/etc/board.d/02_network @@ -97,11 +97,11 @@ realtek_setup_macs() ;; esac - ucidef_set_interface_macaddr "lan" $lan_mac - ucidef_set_bridge_mac "$lan_mac" - ucidef_set_network_device_mac eth0 $lan_mac + [ -n "$lan_mac" ] && ucidef_set_interface_macaddr "lan" $lan_mac + [ -n "$lan_mac" ] && ucidef_set_bridge_mac "$lan_mac" + [ -n "$lan_mac" ] && ucidef_set_network_device_mac eth0 $lan_mac - [ -z "$lan_mac_start" ] && lan_mac_start=$lan_mac + [ -z "$lan_mac_start" -a -n "$lan_mac" ] && lan_mac_start=$lan_mac realtek_setup_macs_lan "$lan_list" "$lan_mac_start" "$lan_mac_end" [ -n "$label_mac" ] && ucidef_set_label_macaddr $label_mac |