Merge "Remove the emulator special case from the "adb root" code."
This commit is contained in:
commit
70385f3017
1 changed files with 0 additions and 7 deletions
|
|
@ -108,13 +108,6 @@ static bool should_drop_privileges() {
|
||||||
#if defined(ALLOW_ADBD_ROOT)
|
#if defined(ALLOW_ADBD_ROOT)
|
||||||
char value[PROPERTY_VALUE_MAX];
|
char value[PROPERTY_VALUE_MAX];
|
||||||
|
|
||||||
// The emulator is never secure, so don't drop privileges there.
|
|
||||||
// TODO: this seems like a bug --- shouldn't the emulator behave like a device?
|
|
||||||
property_get("ro.kernel.qemu", value, "");
|
|
||||||
if (strcmp(value, "1") == 0) {
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
|
|
||||||
// The properties that affect `adb root` and `adb unroot` are ro.secure and
|
// The properties that affect `adb root` and `adb unroot` are ro.secure and
|
||||||
// ro.debuggable. In this context the names don't make the expected behavior
|
// ro.debuggable. In this context the names don't make the expected behavior
|
||||||
// particularly obvious.
|
// particularly obvious.
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue