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

Add necessary permissions for packaged applications (fixes #365) #398

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

m417z
Copy link

@m417z m417z commented May 25, 2024

I didn't do extensive testing, and not sure about any possible regressions caused by switching from Option 2 to Option 1, but from a quick test, adding explicit permissions for "All Application Packages", "All Restricted Application Packages" makes Debugview++ work with UWP apps.

Fixes #365.

@m417z
Copy link
Author

m417z commented May 25, 2024

I looked at how Sysinternals DebugView does that. It does the following:

ConvertStringSecurityDescriptorToSecurityDescriptorA(
  "D:(A;;GRGWGX;;;WD)(A;;GA;;;SY)(A;;GA;;;BA)(A;;GRGWGX;;;AN)(A;;GRGWGX;;;RC)(A;;GRGWGX;;;S-1-15-2-1)S:(ML;;NW;;;LW)",
  SDDL_REVISION_1,
  ...
)

Since DebugView is the de facto standard tool for debug logging, you might want to align DebugView++ to use the exact same security descriptor.

@janwilmans
Copy link
Member

Thank you!! That is very interesting, is there any more you can tell me about what this does, and what it is doing? I am not familiar with this at all.

@m417z
Copy link
Author

m417z commented May 25, 2024

Here are a couple of references:

A process within an AppContainer runs with an Integrity Level of low, which effectively means it has no access to almost everything, as the default integrity level of objects (such as files) is Medium.

My change, and DebugView's method (S-1-15-2-1 representing "All Application Packages") set permissions such that the relevant objects are accessible to AppContainer processes.

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.

OutputDebugString and UWP (Windows 10)
2 participants