[riot-notifications] [RIOT-OS/RIOT] gnrc_icmpv6_echo: use gnrc_netif_hdr_set_netif() (#11916)

Martine Lenders notifications at github.com
Fri Jul 26 16:45:35 CEST 2019


miri64 commented on this pull request.



>              data->hostname = arg;
 #ifdef MODULE_SOCK_DNS
             if (sock_dns_query(data->hostname, &data->host, AF_INET6) == 0) {
                 continue;
             }
 #endif
-            data->iface = ipv6_addr_split_iface(data->hostname);
-            if (data->iface < KERNEL_PID_UNDEF) {
+            iface = ipv6_addr_split_iface(data->hostname);
+            if (iface > KERNEL_PID_UNDEF) {

As you can see, this was just adapted from the existing code. However, looking at other calls of `ipv6_addr_split_iface()` backs you. So I'll provide a piggy-back fix for that.

-- 
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/11916#discussion_r307775878
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190726/7b5ebdc8/attachment-0001.htm>


More information about the notifications mailing list