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

Source IP address for DHCP renewal messages is unset #16012

Closed
pfl opened this issue May 8, 2019 · 2 comments
Closed

Source IP address for DHCP renewal messages is unset #16012

pfl opened this issue May 8, 2019 · 2 comments
Labels
area: Networking bug The issue is a bug, or the PR is fixing a bug
Milestone

Comments

@pfl
Copy link
Collaborator

pfl commented May 8, 2019

The source address in unicast DHCPv4 Request packets was found out
to be all zeros address 0.0.0.0. This address is only acceptable if
the destination is a multicast one, where the host in question is
acquiring a DHCP address lease. This is true for the DHCP Discover
and the initial DHCP Request message from the client towards the
server. As subsequent DHCP Request renewal messages are sent as
unicast to the server, the server will drop such packets. As Linux user space never sees the DHCP Request messages sent with source address 0.0.0.0. It will eventually work, as the Zephyr DHCP client restarts the address acquisition procedure with a DHCP Discover, but depending on the situation and DHCP servers involved, Zephyr might loose its address in between or get a new one assigned due to restarting of the DHCP procedure

@pfl pfl added the bug The issue is a bug, or the PR is fixing a bug label May 8, 2019
@pfl
Copy link
Collaborator Author

pfl commented May 8, 2019

Fixed in upstream by #15179
Fixed in 1.14 backport by #15534

@nashif
Copy link
Member

nashif commented May 9, 2019

fix is merged

@nashif nashif closed this as completed May 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Networking bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

No branches or pull requests

3 participants