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

(Suggestion) Automate adding necessary ASLR exclusions #326

Closed
magic-spearmint opened this issue Feb 6, 2023 · 4 comments
Closed

(Suggestion) Automate adding necessary ASLR exclusions #326

magic-spearmint opened this issue Feb 6, 2023 · 4 comments

Comments

@magic-spearmint
Copy link

magic-spearmint commented Feb 6, 2023

Most binaries from the Cygwin project tend to not play nice with Windows' ASLR exploit mitigation features which are enabled system-wide by default. The usual symptom is that they'll immediately and - most importantly - silently fail to run.

Disabling mandatory and bottom-up ASLR for specific executables can be automated using PowerShell's Set-ProcessMitigation command/cmdlet. Windows 10 Home and Pro have PowerShell 5.x (which has the aforementioned cmdlet) installed and enabled by default.

If wanted, I would be more than willing to adapt an existing small PS script I've written for inclusion with WSLtty. I only need to adjust it so that the code is formalized and abides by strict mode, is triggerable by an installer/external process, and supports checking for, creating, and removing necessary overrides on end users' systems.

Screenshot 2023-02-06 140017

@Biswa96
Copy link
Contributor

Biswa96 commented Feb 6, 2023

Would you like to provide some background? For example,

  • what issue are you trying to solve?
  • how to reproduce the issue?
  • should wsltty installer modify the process mitigation settings instead of user?
  • wouldn't that backfire to wsltty and antivirus programs will start to flag wsltty more often?

@mintty
Copy link
Owner

mintty commented Feb 7, 2023

Related to #6 and #239.
Would global disabling also be required as described in #6 (comment) ?
If so, would installation work for a non-admin user?

@mintty
Copy link
Owner

mintty commented Feb 14, 2023

What's the command to adjust the property properly?

@mintty
Copy link
Owner

mintty commented Mar 12, 2023

Closing as duplicate and missing prospect of a remedy.

@mintty mintty closed this as completed Mar 12, 2023
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

3 participants