Skip to content

sixlowpan: various fixes to sixlowpan_print() function [backport 2018.07]#9629

Merged
cladmi merged 2 commits intoRIOT-OS:2018.07-branchfrom
cladmi:2018.07/sixlowpan/fix/print
Jul 24, 2018
Merged

sixlowpan: various fixes to sixlowpan_print() function [backport 2018.07]#9629
cladmi merged 2 commits intoRIOT-OS:2018.07-branchfrom
cladmi:2018.07/sixlowpan/fix/print

Conversation

@cladmi
Copy link
Copy Markdown
Contributor

@cladmi cladmi commented Jul 24, 2018

Backport of #9585

Contribution description

While working with 6Lo more closely again for #8511, I noticed some bugs in the sixlowpan_print() function.

Issues/PRs references

None

miri64 added 2 commits July 24, 2018 16:27
Destination address compression is all in the least significant nibble
of that byte in the IPHC header [[1]].

[1]: https://tools.ietf.org/html/rfc6282#section-3.1.1

(cherry picked from commit ecfff3e)
We had bad experiences with those in the past when used with newlib-nano
;-)

(cherry picked from commit c6d4605)
@cladmi cladmi added Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) GNRC Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 24, 2018
@cladmi cladmi requested a review from cgundogan July 24, 2018 14:27
@cladmi
Copy link
Copy Markdown
Contributor Author

cladmi commented Jul 24, 2018

I did not re-tested this as I also did not tested the first one. Just created it with the backport script.

@cladmi cladmi merged commit e949655 into RIOT-OS:2018.07-branch Jul 24, 2018
@cladmi cladmi deleted the 2018.07/sixlowpan/fix/print branch July 25, 2018 12:50
@cladmi cladmi added this to the Release 2018.07 milestone Aug 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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 Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants