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.

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

ipv6_addr: provide fix for off-by-x error #6961

Merged
merged 2 commits into from
Apr 26, 2017

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Apr 25, 2017

Provides a fix and test cases for an off-by-one (actually an off-by-2 and off-by-4) error in the string-to-ipv6-address-conversion.

@miri64 miri64 requested a review from kaspar030 April 25, 2017 17:21
@miri64 miri64 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: network Area: Networking labels Apr 25, 2017
@miri64 miri64 added this to the Release 2017.04 milestone Apr 25, 2017
Copy link
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK.

@miri64
Copy link
Member Author

miri64 commented Apr 25, 2017

Backport provided at #6962.

@miri64 miri64 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Apr 25, 2017
@miri64 miri64 merged commit 92fe842 into RIOT-OS:master Apr 26, 2017
@miri64 miri64 deleted the ipv6_addr/fix/off-by-one branch April 26, 2017 05:57
@emmanuelsearch emmanuelsearch removed the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Apr 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking 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.

None yet

3 participants