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

adb daemon: Don't drop privileges #11

Open
wants to merge 1 commit into
base: halium-7.1
Choose a base branch
from

Conversation

fredldotme
Copy link

Running adbd as root via the 'adb root' command causes
adbd to stop and not respawn, so avoid dropping
privileges altogether in favor of running as root.

@Vince1171
Copy link
Member

@fredldotme do you know if htere is any reason why this haven't been merged yet?

@fredldotme
Copy link
Author

@fredldotme do you know if htere is any reason why this haven't been merged yet?

It effectively is a hack, considering that it completely invalidates the ability to build adb-capable and adb-incapable builds by just switching the breakfast target between userdebug and user.

@Vince1171
Copy link
Member

@fredldotme do you know if htere is any reason why this haven't been merged yet?

It effectively is a hack, considering that it completely invalidates the ability to build adb-capable and adb-incapable builds by just switching the breakfast target between userdebug and user.

mmmh so it's not a problem from UT but it could be for other OS?

@fredldotme
Copy link
Author

@fredldotme do you know if htere is any reason why this haven't been merged yet?

It effectively is a hack, considering that it completely invalidates the ability to build adb-capable and adb-incapable builds by just switching the breakfast target between userdebug and user.

mmmh so it's not a problem from UT but it could be for other OS?

It's a problem especially on UT as it does build both breakfast targets and expecting different behaviors. Effectively the change needs to made such that the privilege dropping happens again on user builds.

@Vince1171
Copy link
Member

So we still need it (as it's shipped with most halium 7.1 UT devices) but shouldn't use it?

Running adbd as root via the 'adb root' command causes
adbd to stop and not respawn, so avoid dropping
privileges altogether in favor of running as root.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants