Hide Forgot
Definitely not a big issue, but according to IEEE802, both semicolon (00:11:22:33:44:55) and dashed formats (00-11-22-33-44-55) are valid. So perhaps nmcli's profile editor can allow setting the MAC in dashed format as well, even if it get's converted to ':' immediately.
Sounds like a quick win. Since the standard library functions like ether_aton() don't handle "-" nmcli would probably just quickly search/replace "-" to ":" if the first attempt at ether_aton() failed.
Created attachment 792164 [details] [PATCH] util: allow hyphens in MAC notation - nm_utils_hwaddr_aton_len() The patch allows accepting hyphens (-) in MAC address string representation in nm_utils_hwaddr_aton(_len) function. nmcli uses the functions, so both 'add' and 'edit' nmcli commands will accept such MACs. > Sounds like a quick win. Since the standard library functions like ether_aton() don't handle "-" nmcli would probably just quickly search/replace "-" to ":" if the first attempt at ether_aton() failed. Actually, nmcli doesn't use ether_aton() any more. Instead, it makes use of libnm-util nm_utils_hwaddr_aton() and nm_utils_hwaddr_atoba() functions. So I updated the libnm-util's code. ether_aton() is used on several places in NM, mainly for reading MAC from ifcfg. I'm not sure if we want to allow hyphens there too.
Also pushed to jklimes/rh1002553-mac-hyphens NM upstream branch.
Branch looks good, please merge. THanks!
0e57603e4375ca77cb3c3987865e0293706172c8 (master)
This request was resolved in Red Hat Enterprise Linux 7.0. Contact your manager or support representative in case you have further questions about the request.