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

info: fallback: MPTCP_INFO_FLAG_FALLBACK is never set #6

Merged
merged 1 commit into from
May 6, 2024

Conversation

matttbe
Copy link
Member

@matttbe matttbe commented May 6, 2024

When calling getsockopt(MPTCP_INFO), MPTCP_INFO_FLAG_FALLBACK will never be set. Instead the syscall will return an error, for both the client and the server side.

MPTCP_INFO_FLAG_FALLBACK will only be set when listing MPTCP connections via the Netlink API (hence the confusion).

We can simplify the examples now:

  • getsockopt(MPTCP_INFO) for both the client and server, kernels >= 5.16
  • getsockopt(SO_PROTOCOL) for the server only, only check the client request, kernels < 5.16

When calling getsockopt(MPTCP_INFO), MPTCP_INFO_FLAG_FALLBACK will never
be set. Instead the syscall will return an error, for both the client
and the server side.

MPTCP_INFO_FLAG_FALLBACK will only be set when listing MPTCP connections
via the Netlink API (hence the confusion).

We can simplify the examples now:
- getsockopt(MPTCP_INFO) for both the client and server, kernels >= 5.16
- getsockopt(SO_PROTOCOL) for the server only, only check the client
  request, kernels < 5.16

Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
@matttbe matttbe requested a review from mjmartineau May 6, 2024 11:03
@matttbe
Copy link
Member Author

matttbe commented May 6, 2024

I suggest merging it now, to stop propagating wrong info. We can fix typo later if needed :)

@matttbe matttbe merged commit e67074d into main May 6, 2024
@matttbe matttbe deleted the fix-fallback branch May 6, 2024 11:04
@matttbe matttbe temporarily deployed to github-pages May 6, 2024 11:04 — with GitHub Actions Inactive
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.

1 participant