Merge pull request RIOT-OS#16584 from jia200x/pr/fix_gnrc_lorawan_mem…

…ory_leak gnrc_netif/lorawan: drop netif header on send
hugueslarrive · Jun 25, 2021 · 05b61e8 · 05b61e8
2 parents 472a8d1 + 5808a16
commit 05b61e8
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/sys/net/gnrc/netif/lorawan/gnrc_netif_lorawan.c b/sys/net/gnrc/netif/lorawan/gnrc_netif_lorawan.c
@@ -311,12 +311,15 @@ static int _send(gnrc_netif_t *netif, gnrc_pktsnip_t *payload)
         dst = gnrc_netif_hdr_get_dst_addr(netif_hdr);
 
         assert(payload->type == GNRC_NETTYPE_NETIF);
-        head = payload->next;
         port = dst[0];
 
         if (netif_hdr->dst_l2addr_len != sizeof(port)) {
             goto end;
         }
+
+        /* Remove the netif hdr snip and point to the MSDU */
+        head = gnrc_pktbuf_remove_snip(payload, payload);
+
     }
     else {
         head = payload;