Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PPL process inside a Server Silo #2020

Open
smallmodel opened this issue Apr 4, 2024 · 2 comments
Open

PPL process inside a Server Silo #2020

smallmodel opened this issue Apr 4, 2024 · 2 comments

Comments

@smallmodel
Copy link

smallmodel commented Apr 4, 2024

Description of the feature, modification, idea or suggestion

Hello,

Would it be possible to ignore PPL process check if the target process is running inside a Server Silo? It would make sense as processes running inside a server silo are isolated and are inferior to host processes.

Proposed implementation details (optional)

There is PsGetThreadSilo/PsGetProcessServerSilo/PsIsProcessInSilo

@smallmodel
Copy link
Author

smallmodel commented Apr 5, 2024

Proposed patch: ppl_silo.patch

It would also prevent the calling process running inside a server silo from interfering with processes from other server silos (or the host silo).

To test with Docker Desktop: docker run --rm -ti --isolation process mcr.microsoft.com/windows/nanoserver:2022 and then terminate the newly created csrss.exe that is running inside a server silo.

@jxy-s
Copy link
Member

jxy-s commented Apr 5, 2024

Thank you for the suggestion and patch. This deserves some careful consideration. If the operating system expects to enforce PPL domination between silos I'm unsure we can accept such a patch. If we are to ignore PPL domination in this circumstance, we need to be very certain with this statement:

processes running inside a server silo are isolated and are inferior to host processes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants