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

Initialzing local addr in UDP. #273

Open
wants to merge 1 commit into
base: next
Choose a base branch
from
Open

Conversation

sagsango
Copy link

In UDP local address is ADDR_ANY,
We are initialzing it to local ip (IPv4).

In UDP local address is ADDR_ANY,
We are initialzing it to local ip.

Signed-off-by: Sagar Singh <[email protected]>
@sagsango
Copy link
Author

This is for IPv4.
For IPv6, Can we do same thing in "svc_dg_store_in6_pktinfo()"

@sagsango
Copy link
Author

sagsango commented Mar 24, 2023

Without above changes NFSv3 udp support is not working.
We want to make sure that above changes are not breaking any existing functionalities for any specific NFS clients.

@sagsango
Copy link
Author

sagsango commented Mar 24, 2023

cc: @pradeepthomas, @Gaurav-Gangalwar

@dang
Copy link
Collaborator

dang commented Sep 19, 2023

IP_PKTINFO is supposed to hold the destination address from the IP packet header, so it should be a local address on this machine, not IN_ADDR_ANY. What system are you using?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants