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

Certain Windows configurations can cause an infinite recursion #5

Open
catb0t opened this issue Dec 14, 2018 · 1 comment
Open

Certain Windows configurations can cause an infinite recursion #5

catb0t opened this issue Dec 14, 2018 · 1 comment

Comments

@catb0t
Copy link

catb0t commented Dec 14, 2018

when UAC is disabled (no asking passwords) but the account isn't an administrator, then windows.elevate recurses forever (as isUserAnAdmin() is never true and the function never crashes) and eventually eats up all the RAM spawning new processes.

Next I'll write the more permanent fix but i'm using a shim one in the path-resolving branch for now.

@catb0t
Copy link
Author

catb0t commented Dec 14, 2018

Actually I think this might happen regardless of UAC for me. On the other hand, my Windows 7 is screwed up and customised to the point where it might be the problem (although I wouldn't think with a separate new user account...)

catb0t added a commit to catb0t/elevate that referenced this issue Jun 18, 2019
- close barneygale#3: relative path expansion
- **temporarily** fix barneygale#5 for windows
- documentation is added to the workarounds
    in elevate_util

This implementation uses an import hook, which
    may not be the cleanest solution possible.

The import hook fixes argparse and similar modules
    seeing and complaining about --_with-elevate-*
    options, if the argument parser is called before
    the re-entrant call to elevate().
catb0t added a commit to catb0t/elevate that referenced this issue Jun 18, 2019
- close barneygale#3: relative path expansion
- **temporarily** fix barneygale#5 for windows
- documentation is added to the workarounds
    in elevate_util

This implementation uses an import hook, which
    may not be the cleanest solution possible.

The import hook fixes argparse and similar modules
    seeing and complaining about --_with-elevate-*
    options, if the argument parser is called before
    the re-entrant call to elevate().
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 a pull request may close this issue.

1 participant