aboutsummaryrefslogtreecommitdiff
path: root/libc/bionic/__bionic_get_shell_path.cpp
diff options
context:
space:
mode:
authorSandeep Patil <sspatil@google.com>2017-08-21 12:17:19 -0700
committerSandeep Patil <sspatil@google.com>2017-08-22 10:17:28 -0700
commit9b1ca569db3e698c3a901720ae6989c876a9e6ee (patch)
tree586e7581ad2e2296aa8b24cb4c5bcf598255bfbb /libc/bionic/__bionic_get_shell_path.cpp
parent611d0de62fc7d23dcdd120c2d3698acc85351f9e (diff)
libc: Add support to allow library calls to find appropriate shell executable for a process
Library calls like system() and popen() invoke the shell executable pointed to by '_PATH_BSHELL' in order to run the command passed into the function. The _PATH_BSHELL points to /system/bin/sh by default and thus breaks any vendor process trying to use system() / popen(), as they are denied access to system shell by selinux. This CL make necessary changes, so the implmentations of system() and popen() can use the appropriate shell (e.g. /vendor/bin/sh for processes running out of /vendor partition). Also, changes the implementation of system() and popen(). Bug: 64832610 Test: Manual, Using a test program running from /system/bin and /vendor/bin to ensure correct shell is being used. Change-Id: Ie7168d69decb1ae98284446ae7db34dec930dc33 Merged-In: Ie7168d69decb1ae98284446ae7db34dec930dc33 Signed-off-by: Sandeep Patil <sspatil@google.com> (cherry picked from commit aa3e32422cc7089b9b15976f2651a58840d95e4f)
Diffstat (limited to 'libc/bionic/__bionic_get_shell_path.cpp')
-rw-r--r--libc/bionic/__bionic_get_shell_path.cpp51
1 files changed, 51 insertions, 0 deletions
diff --git a/libc/bionic/__bionic_get_shell_path.cpp b/libc/bionic/__bionic_get_shell_path.cpp
new file mode 100644
index 000000000..477fa4a1b
--- /dev/null
+++ b/libc/bionic/__bionic_get_shell_path.cpp
@@ -0,0 +1,51 @@
+/*
+ * Copyright (C) 2017 The Android Open Source Project
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * * Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in
+ * the documentation and/or other materials provided with the
+ * distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+ * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
+ * COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
+ * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS
+ * OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED
+ * AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT
+ * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
+
+#include <errno.h>
+#include <string.h>
+#include <sys/cdefs.h>
+#include <unistd.h>
+
+__LIBC_HIDDEN__ static const char* __libc_system_sh = "/system/bin/sh";
+__LIBC_HIDDEN__ static const char* __libc_vendor_sh = "/vendor/bin/sh";
+
+static const char* init_sh_path() {
+ /* look for /system or /vendor prefix */
+ char exe_path[7];
+ ssize_t len = readlink("/proc/self/exe", exe_path, sizeof(exe_path));
+ if (len != -1 && !strncmp(exe_path, __libc_vendor_sh, sizeof(exe_path))) {
+ return __libc_vendor_sh;
+ }
+
+ return __libc_system_sh;
+}
+
+__LIBC_HIDDEN__ extern "C" const char* __bionic_get_shell_path() {
+ static const char* sh_path = init_sh_path();
+ return sh_path;
+}