Skip to content

Conversation

@wmmc88
Copy link
Collaborator

@wmmc88 wmmc88 commented Nov 8, 2025

This pull request updates dependencies and improves compatibility with the latest versions of key libraries and tools for Windows driver development. The most important changes include bumping wdk-related crate versions, switching to the latest released version of cargo-wdk in CI, and updating code to use safer pointer initialization for Win32 API calls.

Dependency updates:

  • Updated all wdk-related workspace dependencies in Cargo.toml to newer versions
  • Bumped windows-sys dependency from 0.52.0 to 0.59.0 in general/echo/kmdf/exe/Cargo.toml for improved API coverage and compatibility.

CI/build improvements:

  • Changed the GitHub Actions workflow to install the latest released version of cargo-wdk instead of pinning to a specific commit, ensuring up-to-date tooling for building and packaging drivers.

Win32 API usage and code safety:

  • Updated FFI calls in general/echo/kmdf/exe/src/main.rs to use std::ptr::null_mut() instead of 0 for pointer arguments, improving correctness and clarity in Win32 API interactions. This affects device handle creation, completion port creation, and overlapped I/O event initialization. [1] [2] [3] [4]

related to microsoft/windows-drivers-rs#562

@wmmc88 wmmc88 self-assigned this Nov 8, 2025
@wmmc88 wmmc88 requested a review from a team November 20, 2025 16:58
leon-xd
leon-xd previously approved these changes Nov 20, 2025
@wmmc88 wmmc88 marked this pull request as ready for review November 20, 2025 20:04
@wmmc88 wmmc88 enabled auto-merge (squash) November 20, 2025 21:10
Co-authored-by: Gurinder Singh <[email protected]>
Signed-off-by: Melvin Wang <[email protected]>
@wmmc88 wmmc88 requested review from gurry and leon-xd December 2, 2025 19:21
@wmmc88 wmmc88 merged commit f8a5baa into microsoft:main Dec 3, 2025
42 checks passed
@wmmc88 wmmc88 deleted the nov-release branch December 3, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants