{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3261","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3261/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3261/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3261/events","html_url":"https://github.com/RIOT-OS/RIOT/issues/3261","id":90938315,"number":3261,"title":"gnrc/ng_netdev_eth: potential dereferencing nullpointer","user":{"login":"PeterKietzmann","id":7765855,"avatar_url":"https://avatars.githubusercontent.com/u/7765855?v=3","gravatar_id":"","url":"https://api.github.com/users/PeterKietzmann","html_url":"https://github.com/PeterKietzmann","followers_url":"https://api.github.com/users/PeterKietzmann/followers","following_url":"https://api.github.com/users/PeterKietzmann/following{/other_user}","gists_url":"https://api.github.com/users/PeterKietzmann/gists{/gist_id}","starred_url":"https://api.github.com/users/PeterKietzmann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PeterKietzmann/subscriptions","organizations_url":"https://api.github.com/users/PeterKietzmann/orgs","repos_url":"https://api.github.com/users/PeterKietzmann/repos","events_url":"https://api.github.com/users/PeterKietzmann/events{/privacy}","received_events_url":"https://api.github.com/users/PeterKietzmann/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/duplicate","name":"duplicate","color":"e11d51"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/minor","name":"minor","color":"f7c6c7"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/NSTF","name":"NSTF","color":"d4c5f9"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/question","name":"question","color":"cc317c"}],"state":"closed","locked":false,"assignee":{"login":"kaspar030","id":4679640,"avatar_url":"https://avatars.githubusercontent.com/u/4679640?v=3","gravatar_id":"","url":"https://api.github.com/users/kaspar030","html_url":"https://github.com/kaspar030","followers_url":"https://api.github.com/users/kaspar030/followers","following_url":"https://api.github.com/users/kaspar030/following{/other_user}","gists_url":"https://api.github.com/users/kaspar030/gists{/gist_id}","starred_url":"https://api.github.com/users/kaspar030/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kaspar030/subscriptions","organizations_url":"https://api.github.com/users/kaspar030/orgs","repos_url":"https://api.github.com/users/kaspar030/repos","events_url":"https://api.github.com/users/kaspar030/events{/privacy}","received_events_url":"https://api.github.com/users/kaspar030/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-06-25T11:45:49Z","updated_at":"2015-06-25T12:21:02Z","closed_at":"2015-06-25T12:20:21Z","body":"In ng_netdev_eth the line  [`if (payload)`](https://github.com/RIOT-OS/RIOT/blob/master/sys/net/link_layer/ng_netdev_eth/ng_netdev_eth.c#L376) indicates that `payload` might be `NULL`. Some lines below [`_addr_set_multicast(hdr->dst, payload)`](https://github.com/RIOT-OS/RIOT/blob/master/sys/net/link_layer/ng_netdev_eth/ng_netdev_eth.c#L399) is called. Inside this function, `payload` is [dereferenced](https://github.com/RIOT-OS/RIOT/blob/master/sys/net/link_layer/ng_netdev_eth/ng_netdev_eth.c#L342). What if `payload` **is** actually `NULL`? I don't know if this is a realistic situation or not. Just close this issue if it's superfluous!"},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/115231755","html_url":"https://github.com/RIOT-OS/RIOT/issues/3261#issuecomment-115231755","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/3261","id":115231755,"user":{"login":"PeterKietzmann","id":7765855,"avatar_url":"https://avatars.githubusercontent.com/u/7765855?v=3","gravatar_id":"","url":"https://api.github.com/users/PeterKietzmann","html_url":"https://github.com/PeterKietzmann","followers_url":"https://api.github.com/users/PeterKietzmann/followers","following_url":"https://api.github.com/users/PeterKietzmann/following{/other_user}","gists_url":"https://api.github.com/users/PeterKietzmann/gists{/gist_id}","starred_url":"https://api.github.com/users/PeterKietzmann/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PeterKietzmann/subscriptions","organizations_url":"https://api.github.com/users/PeterKietzmann/orgs","repos_url":"https://api.github.com/users/PeterKietzmann/repos","events_url":"https://api.github.com/users/PeterKietzmann/events{/privacy}","received_events_url":"https://api.github.com/users/PeterKietzmann/received_events","type":"User","site_admin":false},"created_at":"2015-06-25T12:21:02Z","updated_at":"2015-06-25T12:21:02Z","body":"Yes it is was a duplicate. PS: I linked the regarding code lines"}}