[riot-notifications] [RIOT] net/ng_netdev_eth: added missing NULL ptr check (#3238)

Hauke Petersen notifications at github.com
Tue Jun 23 02:22:05 CEST 2015


When calling `_addr_set_multicast`, payload may be NULL. But `_addr_set_multicast` dereferences `payload` in its first line of implementation. So making sure here, that we only call it, if `payload` is not NULL. 

Maybe it might make sense to move the NULL pointer check into `_addr_set_multicast`? I dont have an opinion here...
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/3238

-- Commit Summary --

  * net/ng_netdev_eth: added missing NULL ptr check

-- File Changes --

    M sys/net/link_layer/ng_netdev_eth/ng_netdev_eth.c (6)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/3238.patch
https://github.com/RIOT-OS/RIOT/pull/3238.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/3238
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20150622/65dfb42e/attachment.html>


More information about the notifications mailing list