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

Issue 819262 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

authpolicy-0.0.1-r1078/work/authpolicy-0.0.1/authpolicy/authpolicy_unittest.cc:293: Failure

Project Member Reported by mgild@chromium.org, Mar 6 2018

Issue description

Auth policy unit tests are failing for
amd64-generic-incremental
#45142-45196
Betty-incremental
#108-170
Chell-incremental
#4718-4781

Snippet of log that contains the failure.

authpolicy-0.0.1-r1078: [0306/164837:ERROR:process_executor.cc(190)] Seccomp filter blocked a system call
authpolicy-0.0.1-r1078: [0306/164837:ERROR:samba_interface.cc(1535)] Failed to parse preg files
authpolicy-0.0.1-r1078: [0306/164837:INFO:authpolicy.cc(44)] Device policy fetch and parsing failed with code 12
authpolicy-0.0.1-r1078: ../../../../../../../tmp/portage/chromeos-base/authpolicy-0.0.1-r1078/work/authpolicy-0.0.1/authpolicy/authpolicy_unittest.cc:293: Failure
authpolicy-0.0.1-r1078:       Expected: expected_error
authpolicy-0.0.1-r1078:       Which is: 0
authpolicy-0.0.1-r1078: To be equal to: actual_error
authpolicy-0.0.1-r1078:       Which is: 12
 
Cc: ejcaruso@chromium.org ljusten@chromium.org rsorokin@chromium.org zentaro@chromium.org
Labels: OS-Chrome Pri-2 Type-Bug
I'm hitting this in local testing. Interestingly going back even further than Mar 3 still shows this issue so perhaps something changed in a dependency.
Labels: M-67 Chromad
Owner: ljusten@chromium.org
Status: Assigned (was: Untriaged)
Status: Started (was: Assigned)
Taking a look.
Cc: manojgupta@chromium.org
+manojgupta

I've been able to reproduce this issue and found that it's a heap corruption:

*** Error in `/usr/sbin/authpolicy_parser': malloc(): memory corruption: 0x000055d09f03bec0 ***
======= Backtrace: =========
/lib64/libc.so.6(+0x75492)[0x7fed4e249492]
/lib64/libc.so.6(+0x7b611)[0x7fed4e24f611]
/lib64/libc.so.6(+0x7d815)[0x7fed4e251815]
/lib64/libc.so.6(__libc_malloc+0x55)[0x7fed4e253975]
/usr/lib64/libbase-core-395517.so(ShimCppNew+0x22)[0x7fed4e9fa892]
======= Memory map: ========
Aborted

After doing a clean rebuild, the issue was gone, so I'm assuming it's a problem with incremental builds only (matches your observation of failing tryjobs in the description).

I'm wondering whether it could be related to  https://crbug.com/807685  and https://chromium-review.googlesource.com/900684, which swaps out the memory allocator in ASAN builds. Uprev'ing the libchrome.ebuild doesn't cause stuff to rebuild that depends on libchrome, so maybe we're ending up in a spot where some packages use the old allocator and some use the new one.

A fix might be to depend on >=chromeos-base/libchrome-395517-r22 (although I'm not sure how/where to set that since libchrome is a bit non-standard.

WDYT?

Right now, the custom allocator is disabled only for asan builds (use asan && epatch ). I suspect it should be disabled completely irrespective of asan.
Components: Enterprise
Owner: mgild@chromium.org
I tried disabling the allocator, but that just gave me a different error message.

Given that the incremental builds looks green again and that a clean rebuild fixes it, are you OK to close this out?

Comment 8 by mgild@chromium.org, Apr 17 2018

Status: WontFix (was: Started)

Sign in to add a comment