diff options
| author | nicholss <nicholss@chromium.org> | 2015-09-22 12:48:41 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-09-22 19:49:20 +0000 |
| commit | b8a68ccfae9ce2a5f72e7a18e2fe58b69d17008a (patch) | |
| tree | b28f57c0c8f909e68624511b2501cf71fccc314c /remoting | |
| parent | 87cf74758c3fb0a35f1b695c00d0bddd58bfda41 (diff) | |
| download | chromium_src-b8a68ccfae9ce2a5f72e7a18e2fe58b69d17008a.zip chromium_src-b8a68ccfae9ce2a5f72e7a18e2fe58b69d17008a.tar.gz chromium_src-b8a68ccfae9ce2a5f72e7a18e2fe58b69d17008a.tar.bz2 | |
Fixing an error when service chrome-remote-desktop commands in user mode.
When running service chrome-remote-desktop, the script will now always sudo
as the user running the service command. This loads the environment variables
that service lacks. To replicate the orginal error, run
`service chrome-remoting-desktop stop` as a user and note the error python
throws when attempting to read in HOME from the env.
R=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/1358763005
Cr-Commit-Position: refs/heads/master@{#350214}
Diffstat (limited to 'remoting')
| -rwxr-xr-x | remoting/host/installer/linux/debian/chrome-remote-desktop.init | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/remoting/host/installer/linux/debian/chrome-remote-desktop.init b/remoting/host/installer/linux/debian/chrome-remote-desktop.init index 2fc0d0d..a36d239 100755 --- a/remoting/host/installer/linux/debian/chrome-remote-desktop.init +++ b/remoting/host/installer/linux/debian/chrome-remote-desktop.init @@ -62,11 +62,7 @@ run_and_ignore_error() { shift set +e - if [ "$(whoami)" = "$user" ]; then - "$@" - else - sudo -u "$user" $login_options "$@" - fi + sudo -u "$user" $login_options "$@" } do_start() { |