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

fix: implement workaround for root privilege issues on macOS #73

Merged
merged 11 commits into from
May 25, 2023
Merged

fix: implement workaround for root privilege issues on macOS #73

merged 11 commits into from
May 25, 2023

Conversation

kernel-dev
Copy link
Contributor

Before, users were expected to manually launch the binary contained inside of the app bundle with sudo.

This PR proposes an implementation, written in C, which explicitly asks for the user to input their password in order for the application to restart itself with root privileges. It manually overloads the CFBundleExecutable and CFBundleName identifiers inside of the bundle manifest to point to the compiled binary of this elevation fix.

Special thanks to @dhinakg for helping me debug a syntactical issue for osascript -e.

src/main_elevate.c Outdated Show resolved Hide resolved
src/main_elevate.c Outdated Show resolved Hide resolved
src/main_elevate.c Outdated Show resolved Hide resolved
@moonshadow565 moonshadow565 merged commit 910f2ed into LeagueToolkit:master May 25, 2023
@kernel-dev kernel-dev deleted the macos_elevation_fix branch May 25, 2023 22:30
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.

None yet

2 participants