summaryrefslogtreecommitdiffstats
path: root/blimp/engine
diff options
context:
space:
mode:
authormaniscalco <maniscalco@chromium.org>2015-11-05 15:36:11 -0800
committerCommit bot <commit-bot@chromium.org>2015-11-05 23:37:52 +0000
commitcebd990f8e52b0420e5c8d096d4df1481e906c96 (patch)
treec0f3a700230b7f5ddea40b39c8f36914f816aff1 /blimp/engine
parent31111f33ab1914334ded0fc0560905fa63de8e04 (diff)
downloadchromium_src-cebd990f8e52b0420e5c8d096d4df1481e906c96.zip
chromium_src-cebd990f8e52b0420e5c8d096d4df1481e906c96.tar.gz
chromium_src-cebd990f8e52b0420e5c8d096d4df1481e906c96.tar.bz2
Remove setuid bit from blimp's chrome-sandbox
Now that we've udpated our environment to allow non-privileged users to CLONE_NEWUSER, we no longer need a setuid sandbox. BUG=551140 Review URL: https://codereview.chromium.org/1427523006 Cr-Commit-Position: refs/heads/master@{#358176}
Diffstat (limited to 'blimp/engine')
-rw-r--r--blimp/engine/Dockerfile5
1 files changed, 0 insertions, 5 deletions
diff --git a/blimp/engine/Dockerfile b/blimp/engine/Dockerfile
index 65e985a..21a4d22 100644
--- a/blimp/engine/Dockerfile
+++ b/blimp/engine/Dockerfile
@@ -17,11 +17,6 @@ ADD * /engine/
RUN mv /engine/chrome_sandbox /engine/chrome-sandbox
RUN chown -R blimp_user /engine
-# TODO(sriramsr): Under Docker on GCE, the sandbox fails unless it's setuid
-# (crbug.com/551140).
-RUN chown root /engine/chrome-sandbox
-RUN chmod 4755 /engine/chrome-sandbox
-
USER blimp_user
WORKDIR "/engine"