diff options
author | Zach Reizner <zachr@google.com> | 2020-05-12 17:35:36 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-05-16 02:19:37 +0000 |
commit | 73665866a047c4602ab25ae6e20ce1f98f1f8e6e (patch) | |
tree | 5e86ede9bcc8b673e228e0b91058623ae841e2d5 /vm_control | |
parent | 7ea121b916bba02c87d61ba35e38c8bd27b17163 (diff) | |
download | crosvm-73665866a047c4602ab25ae6e20ce1f98f1f8e6e.tar crosvm-73665866a047c4602ab25ae6e20ce1f98f1f8e6e.tar.gz crosvm-73665866a047c4602ab25ae6e20ce1f98f1f8e6e.tar.bz2 crosvm-73665866a047c4602ab25ae6e20ce1f98f1f8e6e.tar.lz crosvm-73665866a047c4602ab25ae6e20ce1f98f1f8e6e.tar.xz crosvm-73665866a047c4602ab25ae6e20ce1f98f1f8e6e.tar.zst crosvm-73665866a047c4602ab25ae6e20ce1f98f1f8e6e.zip |
msg_socket: remove MsgOnSocket impl for RawFd, an alias for i32
This implementation makes it impossible to use i32 as ordinary data in a MsgOnSocket implementation. It is also error prone because i32 is the default type for integer literals in Rust, and #[derive(MsgOnSocket)] on structs with an i32 would suddenly be transmitting a RawFd. This change also replaces the uses of RawFd's MsgOnSocket impl for one based on File. TEST=cargo build BUG=None Change-Id: I9da392e34e62fe61f773f24d656dbcbf4716f1a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/2197777 Tested-by: Zach Reizner <zachr@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Zach Reizner <zachr@chromium.org> Reviewed-by: Daniel Verkamp <dverkamp@chromium.org>
Diffstat (limited to 'vm_control')
-rw-r--r-- | vm_control/src/lib.rs | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/vm_control/src/lib.rs b/vm_control/src/lib.rs index a8535f8..81a11d3 100644 --- a/vm_control/src/lib.rs +++ b/vm_control/src/lib.rs @@ -19,7 +19,7 @@ use std::os::unix::io::{AsRawFd, FromRawFd, RawFd}; use libc::{EINVAL, EIO, ENODEV}; use kvm::{IrqRoute, IrqSource, Vm}; -use msg_socket::{MsgOnSocket, MsgReceiver, MsgResult, MsgSender, MsgSocket}; +use msg_socket::{MsgError, MsgOnSocket, MsgReceiver, MsgResult, MsgSender, MsgSocket}; use resources::{Alloc, GpuMemoryDesc, MmioType, SystemAllocator}; use sys_util::{error, Error as SysError, EventFd, GuestAddress, MemoryMapping, MmapError, Result}; @@ -53,13 +53,16 @@ impl MsgOnSocket for MaybeOwnedFd { 1usize } unsafe fn read_from_buffer(buffer: &[u8], fds: &[RawFd]) -> MsgResult<(Self, usize)> { - let (fd, size) = RawFd::read_from_buffer(buffer, fds)?; - let file = File::from_raw_fd(fd); + let (file, size) = File::read_from_buffer(buffer, fds)?; Ok((MaybeOwnedFd::Owned(file), size)) } - fn write_to_buffer(&self, buffer: &mut [u8], fds: &mut [RawFd]) -> MsgResult<usize> { - let fd = self.as_raw_fd(); - fd.write_to_buffer(buffer, fds) + fn write_to_buffer(&self, _buffer: &mut [u8], fds: &mut [RawFd]) -> MsgResult<usize> { + if fds.is_empty() { + return Err(MsgError::WrongFdBufferSize); + } + + fds[0] = self.as_raw_fd(); + Ok(1) } } |