Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sixlowpan: iphc: remove dispatch and assign result to pkt #4539

Merged
merged 1 commit into from
Dec 22, 2015
Merged

sixlowpan: iphc: remove dispatch and assign result to pkt #4539

merged 1 commit into from
Dec 22, 2015

Conversation

cgundogan
Copy link
Member

the result of gnrc_pktbuf_remove_snip() needs to be assigned to pkt. Otherwise, I receive a hard fault as described in #4462 (comment)

@cgundogan cgundogan added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: network Area: Networking Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch labels Dec 22, 2015
@cgundogan cgundogan added this to the Release 2015.12 milestone Dec 22, 2015
@cgundogan cgundogan added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Dec 22, 2015
@miri64
Copy link
Member

miri64 commented Dec 22, 2015

ACK and go, when Travis is happy.

@cgundogan
Copy link
Member Author

travis approves - GO

cgundogan added a commit that referenced this pull request Dec 22, 2015
@cgundogan
Merge pull request #4539 from cgundogan/pr/gnrc_sixlowpan/remove_iphc…
7d6addc 
…_dispatch

sixlowpan: iphc: remove dispatch and assign result to pkt
@cgundogan cgundogan merged commit 7d6addc into RIOT-OS:master Dec 22, 2015
@cgundogan cgundogan deleted the pr/gnrc_sixlowpan/remove_iphc_dispatch branch December 22, 2015 20:35
@cgundogan
Copy link
Member Author

I will provide a backport as soon as #4485 was backported (otherwise I get conflicts)

@cgundogan cgundogan mentioned this pull request Dec 23, 2015
Merged
@cgundogan cgundogan removed the Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch label Dec 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@miri64 miri64

Labels
Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
Release 2015.12
Development

Successfully merging this pull request may close these issues.
2 participants
@cgundogan @miri64