diff options
| author | Sandeep Patil <sspatil@google.com> | 2017-08-21 12:17:19 -0700 |
|---|---|---|
| committer | Sandeep Patil <sspatil@google.com> | 2017-08-22 10:17:28 -0700 |
| commit | 9b1ca569db3e698c3a901720ae6989c876a9e6ee (patch) | |
| tree | 586e7581ad2e2296aa8b24cb4c5bcf598255bfbb /libc/bionic/__bionic_get_shell_path.cpp | |
| parent | 611d0de62fc7d23dcdd120c2d3698acc85351f9e (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.cpp | 51 |
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; +} |
