Skip to content

Commit

Permalink
Update windows crate dependency to 0.56 (#88)
Browse files Browse the repository at this point in the history
Minor change for sudo in that `#[must_use]` is now applied to `RPC_STATUS`. This helps to catch cases in which the error code is accidentally ignored. In this particular case, it can safely be ignored.
  • Loading branch information
kennykerr authored Jun 5, 2024
1 parent d67aaec commit 0082369
Show file tree
Hide file tree
Showing 3 changed files with 55 additions and 24 deletions.
75 changes: 53 additions & 22 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ embed-manifest = "1.4"
which = "6.0"
win_etw_provider = "0.1.8"
win_etw_macros = "0.1.8"
windows = "0.54"
windows = "0.56"
windows-registry = "0.1"
winres = "0.1"

Expand Down
2 changes: 1 addition & 1 deletion sudo/src/rpc_bindings_client.rs
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ pub fn rpc_client_setup(endpoint: &CStr) -> RPC_STATUS {
std::ptr::addr_of_mut!(client_sudo_rpc_ClientIfHandle),
);
// Don't forget to free the previously allocated string before potentially returning. :)
RpcStringFreeA(&mut string_binding);
_ = RpcStringFreeA(&mut string_binding);
if status != RPC_S_OK {
return status;
}
Expand Down

0 comments on commit 0082369

Please sign in to comment.