Skip to content

gnrc_ipv6_nib: release packages for forwarding error cases#8459

Merged
smlng merged 1 commit intoRIOT-OS:masterfrom
miri64:gnrc_ipv6_nib/fix/release-fwderror
Jan 26, 2018
Merged

gnrc_ipv6_nib: release packages for forwarding error cases#8459
smlng merged 1 commit intoRIOT-OS:masterfrom
miri64:gnrc_ipv6_nib/fix/release-fwderror

Conversation

@miri64
Copy link
Copy Markdown
Member

@miri64 miri64 commented Jan 26, 2018

Contribution description

This fixes the behavior for the NIB in case of errors. For some error cases in the forwarding look-up the packet wasn't released. This is fixed with that

Issues/PRs references

Fixes #8457 (at least the test).

@miri64 miri64 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: network Area: Networking GNRC Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch labels Jan 26, 2018
@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 Jan 26, 2018
@kYc0o
Copy link
Copy Markdown
Contributor

kYc0o commented Jan 26, 2018

Test now succeed. Test ACK.

Copy link
Copy Markdown
Member

@smlng smlng left a comment

Choose a reason for hiding this comment

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

I can also confirm, that tests/gnrc_sock_udp is currently not working master, but with this fix it does.

@smlng
Copy link
Copy Markdown
Member

smlng commented Jan 26, 2018

@miri64 please provide back port.

@smlng smlng merged commit 2ad122e into RIOT-OS:master Jan 26, 2018
@miri64 miri64 deleted the gnrc_ipv6_nib/fix/release-fwderror branch January 27, 2018 06:28
@miri64 miri64 removed the Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch label Jan 27, 2018
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Jan 27, 2018

Backport provided in #8465

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 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.

Unable to send packets with UDP sock

4 participants