Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(370)

Unified Diff: sandbox/linux/seccomp-bpf/bpf_tests_unittest.cc

Issue 289193003: Convert a bunch of BPF_TEST tests to use BPF_TEST_C (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase (demo.cc fix pushed into parent CL) Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sandbox/linux/seccomp-bpf/bpf_tests_unittest.cc
diff --git a/sandbox/linux/seccomp-bpf/bpf_tests_unittest.cc b/sandbox/linux/seccomp-bpf/bpf_tests_unittest.cc
index d83b8eda3e09dae7557467fb5146e9885d75a0ef..bd18412bee62ff2e4a1a4648d324dad444e1537a 100644
--- a/sandbox/linux/seccomp-bpf/bpf_tests_unittest.cc
+++ b/sandbox/linux/seccomp-bpf/bpf_tests_unittest.cc
@@ -22,23 +22,6 @@ namespace sandbox {
namespace {
-ErrorCode EmptyPolicy(SandboxBPF* sandbox, int sysno, void* aux) {
- // |aux| should always be NULL since a type was not specified as an argument
- // to BPF_TEST.
- BPF_ASSERT(NULL == aux);
- if (!SandboxBPF::IsValidSyscallNumber(sysno)) {
- return ErrorCode(ENOSYS);
- } else {
- return ErrorCode(ErrorCode::ERR_ALLOWED);
- }
-}
-
-BPF_TEST(BPFTest, BPFAUXIsNull, EmptyPolicy) {
- // Check that the implicit BPF_AUX argument is NULL when we
- // don't specify a fourth parameter to BPF_TEST.
- BPF_ASSERT(NULL == BPF_AUX);
-}
-
class FourtyTwo {
public:
static const int kMagicValue = 42;

Powered by Google App Engine
This is Rietveld 408576698