]> git.lizzy.rs Git - rust.git/commitdiff
Fix merge conflicts
authorChristian Poveda <christianpoveda@protonmail.com>
Mon, 21 Oct 2019 13:49:49 +0000 (08:49 -0500)
committerChristian Poveda <christianpoveda@protonmail.com>
Mon, 21 Oct 2019 13:49:49 +0000 (08:49 -0500)
1  2 
src/helpers.rs
src/shims/env.rs
src/shims/fs.rs

diff --cc src/helpers.rs
index 4d84106dbe86f7f75413c7e4d92116e9aba3ed59,16091bb242cdd60191d9723549863a9a7cee9eda..c82971810c0212cb1f51a9fe5b4b5998903d2945
@@@ -346,63 -346,67 +347,127 @@@ pub trait EvalContextExt<'mir, 'tcx: 'm
          Ok(())
      }
  
+     /// Sets the last error variable.
+     fn set_last_error(&mut self, scalar: Scalar<Tag>) -> InterpResult<'tcx> {
+         let this = self.eval_context_mut();
+         let errno_place = this.machine.last_error.unwrap();
+         this.write_scalar(scalar, errno_place.into())
+     }
+     /// Gets the last error variable.
+     fn get_last_error(&mut self) -> InterpResult<'tcx, Scalar<Tag>> {
+         let this = self.eval_context_mut();
+         let errno_place = this.machine.last_error.unwrap();
+         this.read_scalar(errno_place.into())?.not_undef()
+     }
+     /// Sets the last OS error using a `std::io::Error`. This function tries to produce the most
+     /// similar OS error from the `std::io::ErrorKind` and sets it as the last OS error.
+     fn set_last_error_from_io_error(&mut self, e: std::io::Error) -> InterpResult<'tcx> {
+         use std::io::ErrorKind::*;
+         let this = self.eval_context_mut();
+         let target = &this.tcx.tcx.sess.target.target;
+         let last_error = if target.options.target_family == Some("unix".to_owned()) {
+             this.eval_libc(match e.kind() {
+                 ConnectionRefused => "ECONNREFUSED",
+                 ConnectionReset => "ECONNRESET",
+                 PermissionDenied => "EPERM",
+                 BrokenPipe => "EPIPE",
+                 NotConnected => "ENOTCONN",
+                 ConnectionAborted => "ECONNABORTED",
+                 AddrNotAvailable => "EADDRNOTAVAIL",
+                 AddrInUse => "EADDRINUSE",
+                 NotFound => "ENOENT",
+                 Interrupted => "EINTR",
+                 InvalidInput => "EINVAL",
+                 TimedOut => "ETIMEDOUT",
+                 AlreadyExists => "EEXIST",
+                 WouldBlock => "EWOULDBLOCK",
+                 _ => throw_unsup_format!("The {} error cannot be transformed into a raw os error", e)
+             })?
+         } else {
+             // FIXME: we have to implement the windows' equivalent of this.
+             throw_unsup_format!("Setting the last OS error from an io::Error is unsupported for {}.", target.target_os)
+         };
+         this.set_last_error(last_error)
+     }
+     /// Helper function that consumes an `std::io::Result<T>` and returns an
+     /// `InterpResult<'tcx,T>::Ok` instead. In case the result is an error, this function returns
+     /// `Ok(-1)` and sets the last OS error accordingly.
+     ///
+     /// This function uses `T: From<i32>` instead of `i32` directly because some IO related
+     /// functions return different integer types (like `read`, that returns an `i64`)
+     fn try_unwrap_io_result<T: From<i32>>(
+         &mut self,
+         result: std::io::Result<T>,
+     ) -> InterpResult<'tcx, T> {
+         match result {
+             Ok(ok) => Ok(ok),
+             Err(e) => {
+                 self.eval_context_mut().set_last_error_from_io_error(e)?;
+                 Ok((-1).into())
+             }
+         }
+     }
++
 +    /// Helper function to read an OsString from a null-terminated sequence of bytes, which is what
 +    /// the Unix APIs usually handle.
 +    fn read_os_string_from_c_string(&mut self, scalar: Scalar<Tag>) -> InterpResult<'tcx, OsString> {
 +        let bytes = self.eval_context_mut().memory.read_c_str(scalar)?;
 +        Ok(bytes_to_os_str(bytes)?.into())
 +    }
 +
 +    /// Helper function to write an OsStr as a null-terminated sequence of bytes, which is what
 +    /// the Unix APIs usually handle.
 +    fn write_os_str_to_c_string(&mut self, os_str: &OsStr, ptr: Pointer<Tag>, size: u64) -> InterpResult<'tcx> {
 +        let bytes = os_str_to_bytes(os_str)?;
 +        let len = bytes.len();
 +        // If `size` is smaller or equal than `bytes.len()`, writing `bytes` plus the required null
 +        // terminator to memory using the `ptr` pointer would cause an overflow.
 +        if size <= len as u64 {
 +            throw_unsup_format!("OsString of length {} is too large for destination buffer of size {}", len, size)
 +        }
 +        let actual_len = (len as u64)
 +            .checked_add(1)
 +            .map(Size::from_bytes)
 +            .ok_or_else(|| err_unsup_format!("OsString of length {} is too large", len))?;
 +        let this = self.eval_context_mut();
 +        this.memory.check_ptr_access(ptr.into(), actual_len, Align::from_bytes(1).unwrap())?;
 +        let buffer = this.memory.get_mut(ptr.alloc_id)?.get_bytes_mut(&*this.tcx, ptr, actual_len)?;
 +        buffer[..len].copy_from_slice(bytes);
 +        // This is ok because the buffer was strictly larger than `bytes`, so after adding the
 +        // null terminator, the buffer size is larger or equal to `bytes.len()`, meaning that
 +        // `bytes` actually fit inside tbe buffer.
 +        buffer[len] = 0;
 +        Ok(())
 +    }
 +}
 +
 +#[cfg(target_os = "unix")]
 +fn os_str_to_bytes<'tcx, 'a>(os_str: &'a OsStr) -> InterpResult<'tcx, &'a [u8]> {
 +    std::os::unix::ffi::OsStringExt::into_bytes(os_str)
 +}
 +
 +#[cfg(target_os = "unix")]
 +fn bytes_to_os_str<'tcx, 'a>(bytes: &'a[u8]) -> InterpResult<'tcx, &'a OsStr> {
 +    Ok(std::os::unix::ffi::OsStringExt::from_bytes(bytes))
 +}
 +
 +// On non-unix platforms the best we can do to transform bytes from/to OS strings is to do the
 +// intermediate transformation into strings. Which invalidates non-utf8 paths that are actually
 +// valid.
 +#[cfg(not(target_os = "unix"))]
 +fn os_str_to_bytes<'tcx, 'a>(os_str: &'a OsStr) -> InterpResult<'tcx, &'a [u8]> {
 +    os_str
 +        .to_str()
 +        .map(|s| s.as_bytes())
 +        .ok_or_else(|| err_unsup_format!("{:?} is not a valid utf-8 string", os_str).into())
 +}
 +
 +#[cfg(not(target_os = "unix"))]
 +fn bytes_to_os_str<'tcx, 'a>(bytes: &'a[u8]) -> InterpResult<'tcx, &'a OsStr> {
 +    let s = std::str::from_utf8(bytes)
 +        .map_err(|_| err_unsup_format!("{:?} is not a valid utf-8 string", bytes))?;
 +    Ok(&OsStr::new(s))
  }
Simple merge
diff --cc src/shims/fs.rs
Simple merge