Skip to content

Commit 951d833

Browse files
committed
net: route: Check NULL value in debug print
In net_route_add(), do not try to print route information if route to neighbor is not found. Coverity-CID: 188172 Fixes zephyrproject-rtos#10091 Signed-off-by: Jukka Rissanen <[email protected]>
1 parent 0a23067 commit 951d833

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

subsys/net/ip/route.c

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -366,13 +366,16 @@ struct net_route_entry *net_route_add(struct net_if *iface,
366366

367367
tmp = net_route_get_nexthop(route);
368368
nbr = net_ipv6_nbr_lookup(iface, tmp);
369-
llstorage = net_nbr_get_lladdr(nbr->idx);
370-
371-
NET_DBG("Removing the oldest route %s via %s [%s]",
372-
net_sprint_ipv6_addr(&route->addr),
373-
net_sprint_ipv6_addr(tmp),
374-
net_sprint_ll_addr(llstorage->addr,
375-
llstorage->len));
369+
if (nbr) {
370+
llstorage = net_nbr_get_lladdr(nbr->idx);
371+
372+
NET_DBG("Removing the oldest route %s "
373+
"via %s [%s]",
374+
net_sprint_ipv6_addr(&route->addr),
375+
net_sprint_ipv6_addr(tmp),
376+
net_sprint_ll_addr(llstorage->addr,
377+
llstorage->len));
378+
}
376379
} while (0);
377380
#endif /* CONFIG_NET_DEBUG_ROUTE */
378381

0 commit comments

Comments
 (0)