linux_loader: improve code quality

`Option<&T>` has the same ABI layout as `*const T`, so we have some room for
improvement where we get more Rust convenience. Further, a bug is fixed where
INVALID_PARAMETER wasn't returned when the BUFFER_SIZE pointer is NULL.

See UEFI 2.10 13.2.2. EFI_LOAD_FILE2_PROTOCOL.LoadFile() for reference.
This commit is contained in:
Philipp Schuster 2023-10-24 11:05:37 +02:00
parent bb380e1948
commit b7f68a50e6
1 changed files with 15 additions and 14 deletions

View File

@ -66,24 +66,20 @@ struct LoadFile2Protocol {
impl LoadFile2Protocol {
fn load_file(
&mut self,
_file_path: *const FfiDevicePath,
_file_path: Option<&FfiDevicePath>,
_boot_policy: bool,
buffer_size: *mut usize,
buffer: *mut c_void,
buffer_size: Option<&mut usize>,
buffer: *mut u8,
) -> Result<()> {
if buffer.is_null() || unsafe { *buffer_size } < self.initrd_data.len() {
unsafe {
let buffer_size = buffer_size.ok_or(uefi::Error::new(Status::INVALID_PARAMETER, ()))?;
if buffer.is_null() || *buffer_size < self.initrd_data.len() {
// Give the caller a hint for the right buffer size.
*buffer_size = self.initrd_data.len();
}
return Err(Status::BUFFER_TOO_SMALL.into());
};
unsafe {
*buffer_size = self.initrd_data.len();
}
let output_slice: &mut [u8] =
unsafe { &mut *slice_from_raw_parts_mut(buffer as *mut u8, *buffer_size) };
unsafe { &mut *slice_from_raw_parts_mut(buffer, self.initrd_data.len()) };
output_slice.copy_from_slice(&self.initrd_data);
@ -98,7 +94,12 @@ unsafe extern "efiapi" fn raw_load_file(
buffer_size: *mut usize,
buffer: *mut c_void,
) -> Status {
this.load_file(file_path, boot_policy, buffer_size, buffer)
this.load_file(
file_path.as_ref(),
boot_policy,
buffer_size.as_mut(),
buffer.cast(),
)
.status()
}