New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 752274 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

3 sandbox_linux_unittests failing on UbsanVptr

Project Member Reported by thakis@chromium.org, Aug 3 2017

Issue description

Namely:
SandboxBPF.SyntheticPolicy
SandboxBPF.UnsafeTrapWithCond
SandboxBPF.ApplyBasicWhitelistPolicy

https://luci-milo.appspot.com/buildbot/chromium.fyi/ClangToTLinuxUBSanVptr%20tester/1447

[ RUN      ] SandboxBPF.SyntheticPolicy
../../sandbox/linux/tests/unit_tests.cc:260: Failure
      Expected: kExpectedValue
      Which is: 42
To be equal to: subprocess_exit_status
      Which is: 1
Actual test failure: ==-10==ERROR: UndefinedBehaviorSanitizer failed to allocate 0x1000 (4096) bytes of InternalScopedBuffer (error code: 3)
==-10==Sanitizer CHECK failed: /b/c/builder/ClangToTLinuxUBSanVptr/src/third_party/llvm/compiler-rt/lib/sanitizer_common/sanitizer_procmaps_common.cc:78 ((proc_self_maps_.len)) > ((0)) (0, 0)


The builder's been red for a long time, so not clear when this started. Started in https://luci-milo.appspot.com/buildbot/chromium.fyi/ClangToTLinuxUBSanVptr%20tester/1370 maybe, but no clear fault. Error message kind of looks like this might be a ubsan runtime thing?
 
Owner: euge...@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 18 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/56c87b628954bff2bae6bf08e6d63447b8fd26d6

commit 56c87b628954bff2bae6bf08e6d63447b8fd26d6
Author: Evgenii Stepanov <eugenis@google.com>
Date: Fri Aug 18 22:53:32 2017

Fix seccomp_bpf tests with ubsan_vptr.

Fix the tests by relaxing test-only bpf filters to allow the system calls used by the vptr checker.
BUG= 752274 
R=rsesek@chromium.org

Change-Id: I9a99a5b1cc8f536d8035d901c6e7d9b556ddc0c6
Reviewed-on: https://chromium-review.googlesource.com/617388
Reviewed-by: Robert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#495722}
[modify] https://crrev.com/56c87b628954bff2bae6bf08e6d63447b8fd26d6/sandbox/linux/integration_tests/bpf_dsl_seccomp_unittest.cc
[modify] https://crrev.com/56c87b628954bff2bae6bf08e6d63447b8fd26d6/sandbox/linux/seccomp-bpf/syscall_unittest.cc
[modify] https://crrev.com/56c87b628954bff2bae6bf08e6d63447b8fd26d6/sandbox/linux/tests/unit_tests.h

Status: Fixed (was: Started)

Sign in to add a comment