deps: Move from winapi to windows-sys

`winapi` is in maintenance mode and the new blessed way to access Windows APIs are the `windows`
and `windows-sys` crates. I don't think any types of `winapi` were exposed in the public API so
I used `windows-sys` since it has much faster compile times.

Author: poliorcetics

Cargo.toml

@@ -21,4 +21,13 @@ cc = "1.0.67"
 lazy_static = "1.4"
 
 [target.'cfg(target_os = "windows")'.dependencies]
-winapi = { version = "0.3.9", features = ["psapi", "memoryapi", "libloaderapi", "processthreadsapi"] }
+windows-sys = { version = "0.45", features = [
+    "Win32_Foundation",
+    "Win32_System_Diagnostics_Debug",
+    "Win32_System_LibraryLoader",
+    "Win32_System_Memory",
+    "Win32_System_ProcessStatus",
+    "Win32_System_SystemServices",
+    "Win32_System_Threading",
+    "Win32_System_SystemInformation",
+] }

src/windows/mod.rs

@@ -4,7 +4,7 @@ use crate::Segment as SegmentTrait;
 use crate::SharedLibrary as SharedLibraryTrait;
 use crate::{Bias, IterationControl, SharedLibraryId, Svma};
 
-use std::ffi::{CStr, OsStr, OsString};
+use std::ffi::{c_char, CStr, OsStr, OsString};
 use std::fmt;
 use std::marker::PhantomData;
 use std::mem;
@@ -13,19 +13,22 @@ use std::ptr;
 use std::slice;
 use std::usize;
 
-use winapi::ctypes::c_char;
-use winapi::shared::minwindef::{HMODULE, MAX_PATH};
-use winapi::um::libloaderapi::{FreeLibrary, LoadLibraryExW, LOAD_LIBRARY_AS_DATAFILE};
-use winapi::um::memoryapi::VirtualQuery;
-use winapi::um::processthreadsapi::GetCurrentProcess;
-use winapi::um::psapi::{
-    EnumProcessModules, GetModuleFileNameExW, GetModuleInformation, MODULEINFO,
-};
-use winapi::um::winnt::{
+use windows_sys::Win32::Foundation::{HINSTANCE, MAX_PATH};
+use windows_sys::Win32::System::Diagnostics::Debug::{
     IMAGE_DEBUG_DIRECTORY, IMAGE_DEBUG_TYPE_CODEVIEW, IMAGE_DIRECTORY_ENTRY_DEBUG,
-    IMAGE_DOS_HEADER, IMAGE_DOS_SIGNATURE, IMAGE_NT_HEADERS, IMAGE_NT_SIGNATURE,
-    IMAGE_SCN_CNT_CODE, IMAGE_SECTION_HEADER, MEMORY_BASIC_INFORMATION, MEM_COMMIT,
+    IMAGE_NT_HEADERS64, IMAGE_SCN_CNT_CODE, IMAGE_SECTION_HEADER,
+};
+use windows_sys::Win32::System::LibraryLoader::{
+    FreeLibrary, LoadLibraryExW, LOAD_LIBRARY_AS_DATAFILE,
+};
+use windows_sys::Win32::System::Memory::{VirtualQuery, MEMORY_BASIC_INFORMATION, MEM_COMMIT};
+use windows_sys::Win32::System::ProcessStatus::{
+    K32EnumProcessModules, K32GetModuleFileNameExW, K32GetModuleInformation, MODULEINFO,
 };
+use windows_sys::Win32::System::SystemServices::{
+    IMAGE_DOS_HEADER, IMAGE_DOS_SIGNATURE, IMAGE_NT_SIGNATURE,
+};
+use windows_sys::Win32::System::Threading::GetCurrentProcess;
 
 // This is 'RSDS'.
 const CV_SIGNATURE: u32 = 0x5344_5352;
@@ -65,7 +68,7 @@ impl<'a> SegmentTrait for Segment<'a> {
 
     #[inline]
     fn len(&self) -> usize {
-        *unsafe { self.section.Misc.VirtualSize() } as usize
+        unsafe { self.section.Misc.VirtualSize as usize }
     }
 }
 
@@ -140,9 +143,9 @@ impl<'a> SharedLibrary<'a> {
         }
     }
 
-    fn nt_headers(&self) -> Option<&IMAGE_NT_HEADERS> {
+    fn nt_headers(&self) -> Option<&IMAGE_NT_HEADERS64> {
         self.dos_header().and_then(|dos_header| {
-            let nt_headers: &IMAGE_NT_HEADERS =
+            let nt_headers: &IMAGE_NT_HEADERS64 =
                 unsafe { &*(self.module_base().offset(dos_header.e_lfanew as isize) as *const _) };
             if nt_headers.Signature == IMAGE_NT_SIGNATURE {
                 Some(nt_headers)
@@ -154,7 +157,7 @@ impl<'a> SharedLibrary<'a> {
 
     fn debug_directories(&self) -> &[IMAGE_DEBUG_DIRECTORY] {
         self.nt_headers().map_or(&[], |nt_headers| {
-            if nt_headers.OptionalHeader.NumberOfRvaAndSizes <= IMAGE_DIRECTORY_ENTRY_DEBUG as u32 {
+            if nt_headers.OptionalHeader.NumberOfRvaAndSizes <= IMAGE_DIRECTORY_ENTRY_DEBUG {
                 return &[];
             }
             let data_dir =
@@ -236,7 +239,7 @@ impl<'a> SharedLibraryTrait for SharedLibrary<'a> {
     fn segments(&self) -> Self::SegmentIter {
         let sections = self.nt_headers().map(|nt_headers| unsafe {
             let base =
-                (nt_headers as *const _ as *const u8).add(mem::size_of::<IMAGE_NT_HEADERS>());
+                (nt_headers as *const _ as *const u8).add(mem::size_of::<IMAGE_NT_HEADERS64>());
             slice::from_raw_parts(
                 base as *const IMAGE_SECTION_HEADER,
                 nt_headers.FileHeader.NumberOfSections as usize,
@@ -260,36 +263,35 @@ impl<'a> SharedLibraryTrait for SharedLibrary<'a> {
         let proc = unsafe { GetCurrentProcess() };
         let mut modules_size = 0;
         unsafe {
-            if EnumProcessModules(proc, ptr::null_mut(), 0, &mut modules_size) == 0 {
+            if K32EnumProcessModules(proc, ptr::null_mut(), 0, &mut modules_size) == 0 {
                 return;
             }
         }
-        let module_count = modules_size / mem::size_of::<HMODULE>() as u32;
+        let module_count = modules_size / mem::size_of::<HINSTANCE>() as u32;
         let mut modules = vec![unsafe { mem::zeroed() }; module_count as usize];
         unsafe {
-            if EnumProcessModules(proc, modules.as_mut_ptr(), modules_size, &mut modules_size) == 0
+            if K32EnumProcessModules(proc, modules.as_mut_ptr(), modules_size, &mut modules_size)
+                == 0
             {
                 return;
             }
         }
 
-        modules.truncate(modules_size as usize / mem::size_of::<HMODULE>());
+        modules.truncate(modules_size as usize / mem::size_of::<HINSTANCE>());
 
         for module in modules {
             unsafe {
-                let mut module_path = vec![0u16; MAX_PATH + 1];
-                let module_path_len = GetModuleFileNameExW(
-                    proc,
-                    module,
-                    module_path.as_mut_ptr(),
-                    MAX_PATH as u32 + 1,
-                ) as usize;
+                // Cast: `MAX_PATH` fits in a u16, on Windows it'll always fit in a usize
+                let mut module_path = vec![0u16; MAX_PATH as usize + 1];
+                let module_path_len =
+                    K32GetModuleFileNameExW(proc, module, module_path.as_mut_ptr(), MAX_PATH + 1)
+                        as usize;
                 if module_path_len == 0 {
                     continue;
                 }
 
                 let mut module_info = mem::zeroed();
-                if GetModuleInformation(
+                if K32GetModuleInformation(
                     proc,
                     module,
                     &mut module_info,
@@ -305,7 +307,7 @@ impl<'a> SharedLibraryTrait for SharedLibrary<'a> {
                 // loaded before.
                 let handle_lock = LoadLibraryExW(
                     module_path.as_ptr(),
-                    ptr::null_mut(),
+                    Default::default(),
                     LOAD_LIBRARY_AS_DATAFILE,
                 );