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

Issue 740353 link

Starred by 0 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Nov 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

bluez metrics_unittests failing on amd64-generic-asan bot

Project Member Reported by vapier@chromium.org, Jul 8 2017

Issue description

starting with 9720.0.0, the amd64-generic-asan bot failed unittests.  9719.0.0 and older passed.

here's the last pass:
https://uberchromegw.corp.google.com/i/chromiumos/builders/amd64-generic-asan/builds/20508

here's the first fail:
https://uberchromegw.corp.google.com/i/chromiumos/builders/amd64-generic-asan/builds/20509

here's the diff:
https://crosland.corp.google.com/log/9719.0.0..9720.0.0

here's the actual error:
bluez-5.44-r98: FAIL: unit/metrics_unittests
bluez-5.44-r98: ============================
bluez-5.44-r98: 
bluez-5.44-r98: ./unit/metrics_unittests: symbol lookup error: /build/amd64-generic/usr/lib64/libinstallattributes-395517.so: undefined symbol: __asan_option_detect_stack_use_after_return

i don't see anything interesting in the logs ...
 
Owner: yunlian@chromium.org
Status: Assigned (was: Unconfirmed)
Yunlian, can you please take a look at this? 
We have not changed the compiler or the libraries, why are we getting this?
it turns out that bluez does not build with asan. So the error happens. When I enable it, I got 6 errors. It also tries to copy the log to 
/tmp/portage/net-wireless/bluez-5.44-r98/temp/asan_logs, which is a wrong location.

I will prepare a patch to enable asan for bluez first.
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/f228d547d27b9583eb9b95e9fc53b204cd79a64c

commit f228d547d27b9583eb9b95e9fc53b204cd79a64c
Author: Yunlian Jiang <yunlian@google.com>
Date: Tue Jul 11 00:33:21 2017

bluez: enable asan for amd64-generic-asan bot.

bluez does not build with asan, but it uses a libraies that builds
with asan, so we get runtime error. This CL enables asan for bluez
on amd64-generic-asan bot.

BUG= chromium:740353 
TEST=bluez builds with asan in amd64-generic-asan bot.

Change-Id: I1d2615f92ac7067d6da931a0b13fae39f5153388
Reviewed-on: https://chromium-review.googlesource.com/565072
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/f228d547d27b9583eb9b95e9fc53b204cd79a64c/net-wireless/bluez/bluez-9999.ebuild

I don't see any "undefined symbol" errors now, but the test still fails.
e.g. https://uberchromegw.corp.google.com/i/chromiumos/builders/amd64-generic-asan/builds/20570

I think the remaining issue is "ERROR: Can't open file: /tmp/portage/net-wireless/bluez-5.44-r99/temp/asan_logs/asan.NNN".

Comment 5 by vapier@chromium.org, Jul 12 2017

there's two parts now ... first, the unittests shouldn't be failing asan, so that should get fixed :).  second, the log_path doesn't work out because of  issue 741336 .

we can re-assign this to a bluez person to figure out why asan is failing and deal with the log path thing in the other bug.
Status: Verified (was: Assigned)
The unittest on amd64-generic-asan is passing.

Sign in to add a comment