Commit: b6b13f2307f5608e6e2278ea6e8f92698aa384cb - external-koush-Superuser (git)

external/koush/Superuser

Commit MetaInfo

• Revision: b6b13f2307f5608e6e2278ea6e8f92698aa384cb (tree)
• Time: 2013-11-16 02:24:32
• Author: Koushik Dutta <koushd@gmai...>
• Commiter: Koushik Dutta

Log Message

Fix superuser when using ART.

Change-Id: I8ef488afe6f7912d8fac205088c3b431a5cb4a35

Change Summary

• modified: Superuser/jni/su/su.c

Incremental Difference

--- a/Superuser/jni/su/su.c
+++ b/Superuser/jni/su/su.c
@@ -631,12 +631,16 @@ int main(int argc, char *argv[]) {
 
     // attempt to use the daemon client if not root,
     // or this is api 18 and adb shell (/data is not readable even as root)
-    if ((geteuid() != AID_ROOT && getuid() != AID_ROOT) || (is_api_18() && getuid() == AID_SHELL)) {
-        // attempt to connect to daemon...
-        LOGD("starting daemon client %d %d", getuid(), geteuid());
-        return connect_daemon(argc, argv);
+    if (is_api_18()) {
+        if (geteuid() != AID_ROOT || getuid() != AID_ROOT) {
+            // attempt to connect to daemon...
+            LOGD("starting daemon client %d %d", getuid(), geteuid());
+            return connect_daemon(argc, argv);
+        }
     }
 
+    LOGD("skipping daemon client %d %d", getuid(), geteuid());
+
     // Sanitize all secure environment variables (from linker_environ.c in AOSP linker).
     /* The same list than GLibc at this point */
     static const char* const unsec_vars[] = {