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

external/koush/Superuser

Commit MetaInfo

• Revision: b27031ea499498d2632583c167a7e6126391756d (tree)
• Time: 2013-11-24 08:02:58
• Author: Kevin Cernekee <cernekee@gmai...>
• Commiter: Kevin Cernekee

Log Message

Add workaround for Samsung set*id restrictions (bug #196)

Change Summary

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

Incremental Difference

--- a/Superuser/jni/su/su.c
+++ b/Superuser/jni/su/su.c
@@ -623,6 +623,28 @@ static int is_api_18() {
   return ver >= 18;
 }
 
+static void fork_for_samsung(void)
+{
+    // Samsung CONFIG_SEC_RESTRICT_SETUID wants the parent process to have
+    // EUID 0, or else our setresuid() calls will be denied.  So make sure
+    // all such syscalls are executed by a child process.
+    int rv;
+
+    switch (fork()) {
+    case 0:
+        return;
+    case -1:
+        PLOGE("fork");
+        exit(1);
+    default:
+        if (wait(&rv) < 0) {
+            exit(1);
+        } else {
+            exit(WEXITSTATUS(rv));
+        }
+    }
+}
+
 int main(int argc, char *argv[]) {
     // start up in daemon mode if prompted
     if (argc == 2 && strcmp(argv[1], "--daemon") == 0) {
@@ -641,6 +663,8 @@ int main(int argc, char *argv[]) {
 
     LOGD("skipping daemon client %d %d", getuid(), geteuid());
 
+    fork_for_samsung();
+
     // 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[] = {