squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Fix potential integer overflow in SGX memory range calculation.

Browse source 
Thanks to Eduard Marin and David Oswald at the University of Burmingham,
and Jo Van Bulck at KU Leuven for discovering this issue.
This commit is contained in:
Jethro Beekman 2019-05-02 18:15:44 -07:00
parent 08bfe16129
commit 1dc4a38b0e
2 changed files with 17 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/libstd/sys/sgx/abi/mem.rs
View file

@ -27,6 +27,8 @@ pub fn image_base() -> u64 {
}
/// Returns `true` if the specified memory range is in the enclave.
///
/// `p + len` must not overflow.
#[unstable(feature = "sgx_platform", issue = "56975")]
pub fn is_enclave_range(p: *const u8, len: usize) -> bool {
let start = p as u64;
@ -36,6 +38,8 @@ pub fn is_enclave_range(p: *const u8, len: usize) -> bool {
}
/// Returns `true` if the specified memory range is in userspace.
///
/// `p + len` must not overflow.
#[unstable(feature = "sgx_platform", issue = "56975")]
pub fn is_user_range(p: *const u8, len: usize) -> bool {
let start = p as u64;

12
src/libstd/sys/sgx/abi/usercalls/alloc.rs
View file

@ -85,8 +85,10 @@ pub unsafe trait UserSafe {
///
/// * the pointer is not aligned.
/// * the pointer is null.
/// * the pointed-to range does not fit in the address space.
/// * the pointed-to range is not in user memory.
unsafe fn from_raw_sized(ptr: *mut u8, size: usize) -> NonNull<Self> {
assert!(ptr.wrapping_add(size) >= ptr);
let ret = Self::from_raw_sized_unchecked(ptr, size);
Self::check_ptr(ret);
NonNull::new_unchecked(ret as _)
@ -268,6 +270,7 @@ impl<T> User<[T]> where [T]: UserSafe {
///
/// * The pointer is not aligned
/// * The pointer is null
/// * The pointed-to range does not fit in the address space
/// * The pointed-to range is not in user memory
pub unsafe fn from_raw_parts(ptr: *mut T, len: usize) -> Self {
User(NonNull::new_userref(<[T]>::from_raw_sized(ptr as _, len * mem::size_of::<T>())))
@ -372,6 +375,7 @@ impl<T> UserRef<[T]> where [T]: UserSafe {
///
/// * The pointer is not aligned
/// * The pointer is null
/// * The pointed-to range does not fit in the address space
/// * The pointed-to range is not in user memory
pub unsafe fn from_raw_parts<'a>(ptr: *const T, len: usize) -> &'a Self {
&*(<[T]>::from_raw_sized(ptr as _, len * mem::size_of::<T>()).as_ptr() as *const Self)
@ -389,6 +393,7 @@ impl<T> UserRef<[T]> where [T]: UserSafe {
///
/// * The pointer is not aligned
/// * The pointer is null
/// * The pointed-to range does not fit in the address space
/// * The pointed-to range is not in user memory
pub unsafe fn from_raw_parts_mut<'a>(ptr: *mut T, len: usize) -> &'a mut Self {
&mut*(<[T]>::from_raw_sized(ptr as _, len * mem::size_of::<T>()).as_ptr() as *mut Self)
@ -552,10 +557,11 @@ impl UserRef<super::raw::ByteBuffer> {
/// enclave memory.
///
/// # Panics
/// This function panics if:
/// This function panics if, in the user `ByteBuffer`:
///
/// * The pointer in the user `ByteBuffer` is null
/// * The pointed-to range in the user `ByteBuffer` is not in user memory
/// * The pointer is null
/// * The pointed-to range does not fit in the address space
/// * The pointed-to range is not in user memory
pub fn copy_user_buffer(&self) -> Vec<u8> {
unsafe {
let buf = self.to_enclave();