Skip to content
This repository was archived by the owner on Jan 28, 2023. It is now read-only.

Create HAX device securely #421

Merged
merged 2 commits into from
Dec 28, 2021
Merged

Create HAX device securely #421

merged 2 commits into from
Dec 28, 2021

Conversation

wcwang
Copy link
Contributor

@wcwang wcwang commented Dec 8, 2021

Create HAX device with the secure function to strengthen the security
level of access.

  • Replace the device object creation function by IoCreateDeviceSecure()
  • Restrict the maximum scope of the default security settings of the
    device object to authenticated users can read, write and execute

Signed-off-by: Wenchao Wang [email protected]

@wcwang wcwang requested a review from hyuan3 December 8, 2021 12:51
@HaxmCI HaxmCI added the CI:Build Fail CI:Build Fail label Dec 8, 2021
Create HAX device with the secure function to strengthen the security
level of access on Windows.

* Replace the device object creation function by IoCreateDeviceSecure()
* Restrict the maximum scope of the default security settings of the
  device object to authenticated users can read, write and execute

Signed-off-by: Wenchao Wang <[email protected]>
@wcwang wcwang force-pushed the access-au branch 2 times, most recently from 4420426 to c42556e Compare December 9, 2021 11:38
Fix the build errors and warnings for all platforms.

* Add the missing header files
* Move platform-independent macro definitions into the 'core' folder
* Define variables at the beginning of a function

Signed-off-by: Wenchao Wang <[email protected]>
@wcwang wcwang added CI:Build Pass CI:Build Pass and removed CI:Build Fail CI:Build Fail labels Dec 9, 2021
@HaxmCI HaxmCI added CI:Build Fail CI:Build Fail and removed CI:Build Pass CI:Build Pass labels Dec 9, 2021
@wayne-ma wayne-ma added CI:Build Pass CI:Build Pass and removed CI:Build Fail CI:Build Fail labels Dec 13, 2021
@wcwang wcwang merged commit 149d346 into master Dec 28, 2021
@wcwang wcwang deleted the access-au branch December 28, 2021 12:23
@souhaib28
Copy link

.

Copy link

@Ikontv Ikontv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI:Build Pass CI:Build Pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants