[riot-notifications] [RIOT-OS/RIOT] gnrc_ipv6_nib: fix acquire race on gnrc_ipv6_nib_get_next_hop_l2addr() (#16450)

Martine Lenders notifications at github.com
Wed May 5 19:03:18 CEST 2021


@miri64 commented on this pull request.



> @@ -186,6 +186,18 @@ static bool _on_link(const ipv6_addr_t *dst, unsigned *iface)
     return ipv6_addr_is_link_local(dst);
 }
 
+static gnrc_netif_t *_acquire_new_iface(unsigned iface)

Nope, as you can see, I am getting the `netif` pointer from the numerical `iface` below first. Sure, I can do that also before calling `_acquire_new_iface()`, but my thought was, if I do it in any case, why not do it in this function?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16450#discussion_r626744995
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210505/1021f6bd/attachment.htm>


More information about the notifications mailing list