[riot-notifications] [RIOT-OS/RIOT] gnrc_netif_hdr: add setter for netif (#10661)

Martine Lenders notifications at github.com
Fri Jul 26 16:05:00 CEST 2019


miri64 commented on this pull request.



> @@ -173,7 +173,7 @@ static gnrc_pktsnip_t *_recv(gnrc_netif_t *netif)
         goto err;
     }
 
-    ((gnrc_netif_hdr_t *)netif_hdr->data)->if_pid = netif->pid;

Given, that I already found a bug when splitting out #11912 I'd say: never trust my code, even if it is straight forward ^^. I'd like to have the `ping6` adaptation out first (see #11916). If you think the rest is an insta-ACK, I'm happy to refrain from further splitting :-).

-- 
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/10661#discussion_r307757522
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190726/917d78f6/attachment.htm>


More information about the notifications mailing list