mirror of
https://github.com/beerpiss/saekawa.git
synced 2024-11-30 18:24:30 +01:00
streamline section-searching code
This commit is contained in:
parent
c28815b544
commit
a55c2bdd55
@ -1,7 +1,6 @@
|
||||
mod external;
|
||||
|
||||
use std::{
|
||||
ffi::CStr,
|
||||
io::{self, Read},
|
||||
mem::{self},
|
||||
path::Path,
|
||||
@ -289,24 +288,22 @@ pub fn self_update(module: &LibraryHandle) -> Result<bool, SelfUpdateError> {
|
||||
let section_header_offset = (&(*nt_header).OptionalHeader as *const _ as *const u8)
|
||||
.byte_add((*nt_header).FileHeader.SizeOfOptionalHeader as usize)
|
||||
as *const IMAGE_SECTION_HEADER;
|
||||
let section_header = (0..number_of_sections)
|
||||
.find_map(|i| {
|
||||
let header = *section_header_offset
|
||||
.byte_add(mem::size_of::<IMAGE_SECTION_HEADER>() * i as usize);
|
||||
|
||||
for i in 0..number_of_sections {
|
||||
let section_header = *section_header_offset.byte_add(40 * i as usize);
|
||||
let section_name = CStr::from_bytes_until_nul(§ion_header.Name)
|
||||
.unwrap()
|
||||
.to_str()
|
||||
.unwrap();
|
||||
|
||||
if section_name != ".rtext" {
|
||||
continue;
|
||||
if &header.Name == b".rtext\0\0" {
|
||||
Some(header)
|
||||
} else {
|
||||
None
|
||||
}
|
||||
|
||||
let src_addr = module
|
||||
})
|
||||
.ok_or(SelfUpdateError::NoUpdaterCodeSection)?;
|
||||
let section_addr = module
|
||||
.handle()
|
||||
.byte_add(section_header.VirtualAddress as usize)
|
||||
as *mut u8;
|
||||
.byte_add(section_header.VirtualAddress as usize) as *mut u8;
|
||||
let section_size = *section_header.Misc.VirtualSize() as usize;
|
||||
|
||||
let dst_addr = VirtualAlloc(
|
||||
ptr::null_mut(),
|
||||
section_size,
|
||||
@ -320,13 +317,9 @@ pub fn self_update(module: &LibraryHandle) -> Result<bool, SelfUpdateError> {
|
||||
|
||||
debug!(
|
||||
"Copying updater code section from {:p} to {:p}",
|
||||
src_addr, dst_addr
|
||||
section_addr, dst_addr
|
||||
);
|
||||
std::ptr::copy_nonoverlapping(src_addr, dst_addr, section_size);
|
||||
|
||||
let updater_start_address = (replace_with_new_library as PROC)
|
||||
.byte_add(dst_addr as usize)
|
||||
.byte_sub(src_addr as usize);
|
||||
std::ptr::copy_nonoverlapping(section_addr, dst_addr, section_size);
|
||||
|
||||
debug!("Making updater code executable");
|
||||
let mut old_protect = 0u32;
|
||||
@ -353,20 +346,19 @@ pub fn self_update(module: &LibraryHandle) -> Result<bool, SelfUpdateError> {
|
||||
debug!("Allocated heap for updater code at {heap:p}");
|
||||
|
||||
(*heap).module = module.handle();
|
||||
let old = U16CString::from_str_truncate(module_filename);
|
||||
let new = U16CString::from_str_truncate(new_module_filename);
|
||||
std::ptr::copy_nonoverlapping(
|
||||
old.as_ptr(),
|
||||
(*heap).old.as_mut_ptr(),
|
||||
old.as_slice().len(),
|
||||
);
|
||||
std::ptr::copy_nonoverlapping(
|
||||
new.as_ptr(),
|
||||
(*heap).new.as_mut_ptr(),
|
||||
new.as_slice().len(),
|
||||
);
|
||||
for (i, c) in module_filename.encode_utf16().enumerate() {
|
||||
(*heap).old[i] = c;
|
||||
}
|
||||
for (i, c) in new_module_filename.encode_utf16().enumerate() {
|
||||
(*heap).new[i] = c;
|
||||
}
|
||||
|
||||
let updater_start_address = (replace_with_new_library as PROC)
|
||||
.byte_add(dst_addr as usize)
|
||||
.byte_sub(section_addr as usize);
|
||||
|
||||
debug!("Executing updater code at {updater_start_address:p}");
|
||||
|
||||
let handle = CreateThread(
|
||||
ptr::null_mut(),
|
||||
0,
|
||||
@ -386,10 +378,7 @@ pub fn self_update(module: &LibraryHandle) -> Result<bool, SelfUpdateError> {
|
||||
});
|
||||
}
|
||||
|
||||
return Ok(true);
|
||||
}
|
||||
|
||||
Err(SelfUpdateError::NoUpdaterCodeSection)
|
||||
Ok(true)
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user