summaryrefslogtreecommitdiff
path: root/chromium/sandbox/linux/syscall_broker/DEPS
diff options
context:
space:
mode:
authorAllan Sandfeld Jensen <allan.jensen@qt.io>2021-09-16 13:17:20 +0200
committerAllan Sandfeld Jensen <allan.jensen@qt.io>2021-10-04 10:21:03 +0200
commitb6fc11be60033b64b1dc20eeafd706cf742c9e81 (patch)
treeb4676b53a8eb172bd6a3f606d9228d2af17d2e57 /chromium/sandbox/linux/syscall_broker/DEPS
parentcfc20437c0d2979452c545266b126c2d3285e59b (diff)
downloadqtwebengine-chromium-b6fc11be60033b64b1dc20eeafd706cf742c9e81.tar.gz
[Backport] Reland "Reland "Linux sandbox syscall broker: use struct kernel_stat""
This reverts commit ff277a52ece0b216617d770f201ed66955fe70b9. Reason for revert: reland The fix included in the reland is that fstatat64() needs to be allowed in the broker process's seccomp policy. This CL also includes some extra tests that the kernel_stat structures match the layout the kernel expects. Bug: 1164975, 1199431 Test: trogdor Chromebook successfully boots and allows login. Original change's description: > Revert "Reland "Linux sandbox syscall broker: use struct kernel_stat"" > > This reverts commit cffbc4432af79f720ae3c75dff380b853701bd64. > > Reason for revert: https://bugs.chromium.org/p/chromium/issues/detail?id=1199431 > > Original change's description: > > Reland "Linux sandbox syscall broker: use struct kernel_stat" > > > > This reverts commit 23030dc650cdfa22631f25bef937905f27f06a2c. > > > > Original change's description: > > > Revert "Linux sandbox syscall broker: use struct kernel_stat" > > > > > > This reverts commit 784b0fcd8a3ca6bcd3acb9cfd624ec9cbbac2789. > > > > > > Reason for revert: Causing failure in > > > Step "sandbox_linux_unittests" failing on builder "Linux ChromiumOS MSan Tests" > > > See crbug.com/1198480 > > > > > > Original change's description: > > > > Linux sandbox syscall broker: use struct kernel_stat > > > > > > > > The struct stat used in libc is different (in size and field ordering) > > > > from the structure assumed by the Linux kernel. So, when emulating > > > > system calls, we need to use the struct definition the kernel expects. > > > > > > > > This CL adds linux_stat.h that includes definitions of the different > > > > kernel structs. > > > > > > > > Change-Id: I53cad35c2251dff0f6b7ea77528cfa58ef3cab4a > > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2780876 > > > > Commit-Queue: Matthew Denton <mpdenton@chromium.org> > > > > Reviewed-by: Robert Sesek <rsesek@chromium.org> > > > > Cr-Commit-Position: refs/heads/master@{#871767} > > > > > > Change-Id: Icbec38f2103c8424dec79ab1870b97c3e83f9361 > > > No-Presubmit: true > > > No-Tree-Checks: true > > > No-Try: true > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2821812 > > > Auto-Submit: Victor Vianna <victorvianna@google.com> > > > Owners-Override: Victor Vianna <victorvianna@google.com> > > > Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > > > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > > > Cr-Commit-Position: refs/heads/master@{#871882} > > > > Change-Id: I1f39bb5242961474def594ff7dbea52009f2cee4 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2824115 > > Auto-Submit: Matthew Denton <mpdenton@chromium.org> > > Commit-Queue: Matthew Denton <mpdenton@chromium.org> > > Reviewed-by: Robert Sesek <rsesek@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#872812} > > Fixed: 1199431 > Change-Id: Iebfc0c48201bf22ff9c54d8d5c8a43d26a880098 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2830459 > Auto-Submit: Kyle Horimoto <khorimoto@chromium.org> > Commit-Queue: Matthew Denton <mpdenton@chromium.org> > Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> > Reviewed-by: Matthew Denton <mpdenton@chromium.org> > Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> > Owners-Override: Kinuko Yasuda <kinuko@chromium.org> > Cr-Commit-Position: refs/heads/master@{#873173} Change-Id: Ibe6a485070f33489aaa157b51b908c2d23d174d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2848936 Reviewed-by: Robert Sesek <rsesek@chromium.org> Commit-Queue: Matthew Denton <mpdenton@chromium.org> Cr-Commit-Position: refs/heads/master@{#902981}
Diffstat (limited to 'chromium/sandbox/linux/syscall_broker/DEPS')
-rw-r--r--chromium/sandbox/linux/syscall_broker/DEPS3
1 files changed, 2 insertions, 1 deletions
diff --git a/chromium/sandbox/linux/syscall_broker/DEPS b/chromium/sandbox/linux/syscall_broker/DEPS
index c477f7d3639..149c463b068 100644
--- a/chromium/sandbox/linux/syscall_broker/DEPS
+++ b/chromium/sandbox/linux/syscall_broker/DEPS
@@ -1,4 +1,5 @@
include_rules = [
- "+sandbox/linux/system_headers",
"+sandbox/linux/bpf_dsl",
+ "+sandbox/linux/services",
+ "+sandbox/linux/system_headers",
]