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

Fixes for empty and multiple-entry control message buffers. #915

Merged
merged 3 commits into from
Nov 8, 2023

Conversation

sunfishcode
Copy link
Member

  • Fix the handling of empty buffers passed to SendAncillaryBuffer::new and RecvAncillaryBuffer::new.

  • Fix the buffer size computation for multiple messages to include only one copy of the padding for alignment.

@sunfishcode sunfishcode force-pushed the sunfishcode/cmsg-combo branch 3 times, most recently from 2f8a4f1 to ed89d19 Compare November 7, 2023 14:46
 - Fix the handling of empty buffers passed to `SendAncillaryBuffer::new` and
   `RecvAncillaryBuffer::new`.

 - Fix the buffer size computation for multiple messages to include only one
   copy of the padding for alignment.
@sunfishcode sunfishcode force-pushed the sunfishcode/cmsg-combo branch from ed89d19 to 4ef82d1 Compare November 7, 2023 15:43
In particular, this brings in sunfishcode/linux-raw-sys#92, a fix for
the `CMSG_NXTHDR` macro which fixes test failures in this PR.
@sunfishcode sunfishcode merged commit 31dfad1 into main Nov 8, 2023
43 checks passed
@sunfishcode sunfishcode deleted the sunfishcode/cmsg-combo branch November 8, 2023 11:26
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