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

gnrc_ndp.c: parsing of SLLAO in gnrc_ndp_rtr_sol_handle #4520

Merged
merged 1 commit into from
Dec 19, 2015
Merged

gnrc_ndp.c: parsing of SLLAO in gnrc_ndp_rtr_sol_handle #4520

merged 1 commit into from
Dec 19, 2015

Conversation

jfischer-no
Copy link
Contributor

Do not set the link-layer address if no SLLAO was included
into the router solicitation, as proposed in #4499.

 Do not set the link-layer address if no SLLAO was included
 into the router solicitation.
@jfischer-no jfischer-no added the Area: network Area: Networking label Dec 19, 2015
@jfischer-no jfischer-no added this to the Release 2015.12 milestone Dec 19, 2015
@miri64
Copy link
Member

miri64 commented Dec 19, 2015

Now the neighbor cache isn't updated at all. But after reading the RFC again that isn't necessary. Will test this tomorrow.

@jfischer-no
Copy link
Contributor Author

ok, I was a little unsure, and I go to sleep ... 💤

@miri64
Copy link
Member

miri64 commented Dec 19, 2015

ACK, please provide a backport to the release branch.

@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch labels Dec 19, 2015
miri64 added a commit that referenced this pull request Dec 19, 2015
…lao_was_included

gnrc_ndp.c: parsing of SLLAO in gnrc_ndp_rtr_sol_handle
@miri64 miri64 merged commit 9eb367a into RIOT-OS:master Dec 19, 2015
@cgundogan cgundogan removed the Process: needs backport Integration Process: The PR is required to be backported to a release or feature branch label Apr 15, 2016
@jfischer-no jfischer-no deleted the pr@set_lladdr_if_sllao_was_included branch January 23, 2017 09:17
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants