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

Implement RawSocket.readMessage/writeMessage. #46481

Closed

Conversation

robert-ancell
Copy link
Contributor

@robert-ancell robert-ancell commented Jun 28, 2021

Fixes #46328

Some notes on the implementation:

  • A specific case where this feature will be used is in D-Bus.
  • This on RawSocket and not Socket, since when receiving a message you need to decide if using recv or recvmsg.
  • I haven't done any additional wrapping of the file descriptors - they are just left as integers for the API user to interpret. This might either involve passing them to some FFI code, or using some new Dart constructors that can take file descriptors.
  • I've only done an implementation for Linux, though other platforms (Android?) may also make use of these.

@google-cla google-cla bot added the cla: yes label Jun 28, 2021
@robert-ancell robert-ancell force-pushed the socket-send-recv-msg branch 2 times, most recently from ecbf42e to 36618c1 Compare June 28, 2021 00:37
@mraleph
Copy link
Member

mraleph commented Jun 29, 2021

Review will happen on the corresponding Gerrit CL https://dart-review.googlesource.com/c/sdk/+/205067

@mit-mit
Copy link
Member

mit-mit commented Sep 27, 2021

Looks like the gerrit review for this is now https://dart-review.googlesource.com/c/sdk/+/212036

@robert-ancell
Copy link
Contributor Author

@robert-ancell robert-ancell deleted the socket-send-recv-msg branch October 19, 2021 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support sending/receiving file descriptors over sockets
3 participants