Age | Commit message (Collapse) | Author |
|
Normally we don't do this, but enough changes have accumulated that
we're doing a tree-wide one-off update of the name & style.
BUG=chromium:1098010
TEST=`repo upload` works
Change-Id: I208569250e49179362ec9e2343ffc13652f576dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minijail/+/3893890
Tested-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Allen Webb <allenwebb@google.com>
Auto-Submit: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Allen Webb <allenwebb@google.com>
|
|
The way we've been hacking access to the si_syscall member no longer
works with newer glibc versions. Change the code to use si_syscall
directly if it's available, and fallback to the old logic only when
it's not (e.g. for Android compatibility).
Bug: chromium:846398
Test: build & unittests still pass
Change-Id: I71e32f8eaca14168ebbe4a4a8b2eea8235929f3c
|
|
The short name "nr" is most commonly used with syscall numbers, not with
signal numbers. For signal numbers, "sig" is a much more common name.
Change all the signal handling code accordingly to avoid confusion.
As an example, log_sigsys_handler has been logging the signal number when
it really wanted a syscall number because it used "nr" as the variable.
Bug: chromium:846398
Test: build passes
Change-Id: I5da79af9fd3a8565cf9c86b9f97f9a11caa46472
|
|
The existing inline attribute usage is a bit ugly to read, and doesn't
use "best practices" when it comes to style. Add explicit defines for
the rest of the code to use, and add some human readable documentation.
Stylewise: use __ everywhere, and move the unused attribute to the end
to match our existing C++ style.
We convert log_sigsys_handler from the legacy method of declaring unused
parameters (a (void) line) to the cleaner attribute style.
Also drop the unused attribute in forward_signal while we're changing
the line because we are actually using the first argument.
Bug: None
Test: build & unittests still pass
Change-Id: I36e5e7cfe4853645de65a5ade963c2b972bb986e
|
|
Tells debuggerd to attach to the process and send crash data
to logcat.
Bug: 26580739
Change-Id: I9df09f044e56cf250963d4e4cf5959642fe2ee2f
|
|
This requires disabling LDPRELOAD and temporarily disabling
capabilities support.
Reland of https://android-review.googlesource.com/#/c/159755/
with compile fixes. Compile-tested on
aosp_{x86,x86_64,arm,arm64,mips64}-eng.
Bug: 22487289
Change-Id: Ia4530cf09b074aa0a2afe5a5b307ff3c5c5d6c08
|
|
Fails compile on x86_64, arm64, mips64. Failures have been identified, will re-upload on Monday.
This reverts commit b9a322d86635c5b1358af0d46a8be1021f4ddb60.
Change-Id: I14b35a3aae618da4ff108328a499505893c15568
|
|
This requires disabling LDPRELOAD and temporarily disabling
capabilities support.
Bug: 22487289
Change-Id: I27476d09605076b000d302f354e49ab17dc96a93
|