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

[v3.7] mDNS reponder might fail to serve if mDNS resolver is enabled #78146

Closed
jukkar opened this issue Sep 9, 2024 · 0 comments · Fixed by #78148
Closed

[v3.7] mDNS reponder might fail to serve if mDNS resolver is enabled #78146

jukkar opened this issue Sep 9, 2024 · 0 comments · Fixed by #78148
Assignees
Labels
area: Networking bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Milestone

Comments

@jukkar
Copy link
Member

jukkar commented Sep 9, 2024

Depending on number of services configured and if the DNS dispatcher is active, so both responder and resolver are enabled at the same time, the mDNS responder might not get any queries.

This issue is created for backporting 3.7 purposes.

@jukkar jukkar added bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug area: Networking labels Sep 9, 2024
@jukkar jukkar added this to the v3.7.1 milestone Sep 9, 2024
@jukkar jukkar self-assigned this Sep 9, 2024
jukkar added a commit to jukkar/zephyr that referenced this issue Sep 9, 2024
The socket services API has a limitation where the user data is shared
between file descriptors described in the same service.

This can cause problem in DNS dispatcher where each listened socket
needs to have their own dispatcher struct set as user data so that we
can map between dispatcher context and socket. Solve this by always
have a dispatcher table as user data, and then have the actual mapping
done via the dispatcher table when receiving data to the dispatcher socket.

Fixes zephyrproject-rtos#78146

Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
@nashif nashif closed this as completed in caf9fd3 Sep 11, 2024
zephyrbot pushed a commit that referenced this issue Sep 11, 2024
The socket services API has a limitation where the user data is shared
between file descriptors described in the same service.

This can cause problem in DNS dispatcher where each listened socket
needs to have their own dispatcher struct set as user data so that we
can map between dispatcher context and socket. Solve this by always
have a dispatcher table as user data, and then have the actual mapping
done via the dispatcher table when receiving data to the dispatcher socket.

Fixes #78146

Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
(cherry picked from commit caf9fd3)
kartben pushed a commit to kartben/zephyr that referenced this issue Sep 11, 2024
The socket services API has a limitation where the user data is shared
between file descriptors described in the same service.

This can cause problem in DNS dispatcher where each listened socket
needs to have their own dispatcher struct set as user data so that we
can map between dispatcher context and socket. Solve this by always
have a dispatcher table as user data, and then have the actual mapping
done via the dispatcher table when receiving data to the dispatcher socket.

Fixes zephyrproject-rtos#78146

Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
nagelkl pushed a commit to nagelkl/zephyr that referenced this issue Sep 16, 2024
The socket services API has a limitation where the user data is shared
between file descriptors described in the same service.

This can cause problem in DNS dispatcher where each listened socket
needs to have their own dispatcher struct set as user data so that we
can map between dispatcher context and socket. Solve this by always
have a dispatcher table as user data, and then have the actual mapping
done via the dispatcher table when receiving data to the dispatcher socket.

Fixes zephyrproject-rtos#78146

Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
nashif pushed a commit that referenced this issue Sep 23, 2024
The socket services API has a limitation where the user data is shared
between file descriptors described in the same service.

This can cause problem in DNS dispatcher where each listened socket
needs to have their own dispatcher struct set as user data so that we
can map between dispatcher context and socket. Solve this by always
have a dispatcher table as user data, and then have the actual mapping
done via the dispatcher table when receiving data to the dispatcher socket.

Fixes #78146

Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
(cherry picked from commit caf9fd3)
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 priority: medium Medium impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant