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

external/koush/Superuser

Commit MetaInfo

• Revision: a43f033dc9a1472821e57aab76b67b76263c99c4 (tree)
• Time: 2013-11-30 02:17:57
• Author: Koushik Dutta <koushd@gmai...>
• Commiter: Koushik Dutta

Log Message

Merge remote-tracking branch 'cern/multiuser-regression-v1' into cm-11.0

Conflicts:
Superuser/jni/su/su.c

Change-Id: I2474f5048e590592e1baa7f4bb1caf76e1245b74

Change Summary

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

Incremental Difference

--- a/Superuser/jni/su/su.c
+++ b/Superuser/jni/su/su.c
@@ -607,6 +607,28 @@ static int get_api_version() {
   return ver;
 }
 
+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[]) {
     return su_main(argc, argv, 1);
 }
@@ -617,6 +639,8 @@ int su_main(int argc, char *argv[], int need_client) {
         return run_daemon();
     }
 
+    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[] = {