{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4485","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4485/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4485/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4485/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/4485","id":122342016,"number":4485,"title":"gnrc_sixlowpan: Fix IPHC/NHC","user":{"login":"authmillenon","id":675644,"avatar_url":"https://avatars.githubusercontent.com/u/675644?v=3","gravatar_id":"","url":"https://api.github.com/users/authmillenon","html_url":"https://github.com/authmillenon","followers_url":"https://api.github.com/users/authmillenon/followers","following_url":"https://api.github.com/users/authmillenon/following{/other_user}","gists_url":"https://api.github.com/users/authmillenon/gists{/gist_id}","starred_url":"https://api.github.com/users/authmillenon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/authmillenon/subscriptions","organizations_url":"https://api.github.com/users/authmillenon/orgs","repos_url":"https://api.github.com/users/authmillenon/repos","events_url":"https://api.github.com/users/authmillenon/events{/privacy}","received_events_url":"https://api.github.com/users/authmillenon/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/API%20CHANGE","name":"API CHANGE","color":"e11d21"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/backporting-needed","name":"backporting-needed","color":"f7c6c7"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/network","name":"network","color":"bfdadc"},{"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Waiting%20For%20Other%20PR","name":"Waiting For Other PR","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"jfischer-phytec-iot","id":9864546,"avatar_url":"https://avatars.githubusercontent.com/u/9864546?v=3","gravatar_id":"","url":"https://api.github.com/users/jfischer-phytec-iot","html_url":"https://github.com/jfischer-phytec-iot","followers_url":"https://api.github.com/users/jfischer-phytec-iot/followers","following_url":"https://api.github.com/users/jfischer-phytec-iot/following{/other_user}","gists_url":"https://api.github.com/users/jfischer-phytec-iot/gists{/gist_id}","starred_url":"https://api.github.com/users/jfischer-phytec-iot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jfischer-phytec-iot/subscriptions","organizations_url":"https://api.github.com/users/jfischer-phytec-iot/orgs","repos_url":"https://api.github.com/users/jfischer-phytec-iot/repos","events_url":"https://api.github.com/users/jfischer-phytec-iot/events{/privacy}","received_events_url":"https://api.github.com/users/jfischer-phytec-iot/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/milestones/10","html_url":"https://github.com/RIOT-OS/RIOT/milestones/Release%202015.12","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/milestones/10/labels","id":871004,"number":10,"title":"Release 2015.12","description":"This is the next major milestone.","creator":{"login":"OlegHahm","id":1590423,"avatar_url":"https://avatars.githubusercontent.com/u/1590423?v=3","gravatar_id":"","url":"https://api.github.com/users/OlegHahm","html_url":"https://github.com/OlegHahm","followers_url":"https://api.github.com/users/OlegHahm/followers","following_url":"https://api.github.com/users/OlegHahm/following{/other_user}","gists_url":"https://api.github.com/users/OlegHahm/gists{/gist_id}","starred_url":"https://api.github.com/users/OlegHahm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OlegHahm/subscriptions","organizations_url":"https://api.github.com/users/OlegHahm/orgs","repos_url":"https://api.github.com/users/OlegHahm/repos","events_url":"https://api.github.com/users/OlegHahm/events{/privacy}","received_events_url":"https://api.github.com/users/OlegHahm/received_events","type":"User","site_admin":false},"open_issues":20,"closed_issues":251,"state":"open","created_at":"2014-11-18T10:51:58Z","updated_at":"2015-12-19T21:02:00Z","due_on":"2015-12-17T23:00:00Z","closed_at":null},"comments":34,"created_at":"2015-12-15T19:07:04Z","updated_at":"2015-12-21T12:07:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/4485","html_url":"https://github.com/RIOT-OS/RIOT/pull/4485","diff_url":"https://github.com/RIOT-OS/RIOT/pull/4485.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/4485.patch"},"body":"Fixes #4397 (and maybe #4462). Could not test because for some reason I can't send anything right now (even with `txtsnd`).\r\n\r\nHere is the general idea of this fix: the `ipv6` parameter of `gnrc_sixlowpan_iphc_decode()` is replaced with the out-parameter `dec_hdr`. While `gnrc_sixlowpan_iphc_decode()` handles the dereferenced pointer the same as it used to handle `ipv6` `iphc_nhc_udp_decode()` sets it to the UDP header and appends the IPv6 header (the old value of `dec_hdr`) to the UDP header. This way the correct header order should be preserved.\r\n\r\nDepends on #4507."},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/166287053","html_url":"https://github.com/RIOT-OS/RIOT/pull/4485#issuecomment-166287053","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/4485","id":166287053,"user":{"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},"created_at":"2015-12-21T12:07:31Z","updated_at":"2015-12-21T12:07:31Z","body":"I just tested merging in this PR, and I have the same problem as before: CoAP reply arrives garbled. \"ifconfig -iphc\" on the ieee802154 doesn't help.\r\n\r\n(But I now learned that \"led toggle\" via coap is a bad idea over lossy networks ;) )"}}