michaelh/zephyr
1
0
Fork 0
Code Releases Activity

net: fix for assert on enabling socket CAN

Browse source 
Socket CAN does not require interface link address to be
assigned, check is not applicable for socket CAN
interfaces. As this address is NULL for socket CAN interface
it results in assertion.

Signed-off-by: Ramesh Babu B <ramesh.babu.b@intel.com>
This commit is contained in:
Ramesh Babu B 2021-07-30 05:50:12 +05:30 • committed by Christopher Friedt
commit d40d963bef
1 changed files with 9 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

10
subsys/net/ip/net_if.c
View file

@ -3990,7 +3990,15 @@ done:
/* In many places it's assumed that link address was set with
* net_if_set_link_addr(). Better check that now.
*/
NET_ASSERT(net_if_get_link_addr(iface)->addr != NULL);
#if defined(CONFIG_NET_L2_CANBUS_RAW)
if (IS_ENABLED(CONFIG_NET_SOCKETS_CAN) &&
(net_if_l2(iface) == &NET_L2_GET_NAME(CANBUS_RAW))) {
/* CAN does not require link address. */
} else
#endif /* CONFIG_NET_L2_CANBUS_RAW */
{
NET_ASSERT(net_if_get_link_addr(iface)->addr != NULL);
}
net_if_flag_set(iface, NET_IF_UP);