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

Issue 771627 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

DnsConfigServicePosixTest.ChangeConfigMultipleTimes crashes when wifi is enabled

Project Member Reported by mge...@chromium.org, Oct 4 2017

Issue description

This isn't happening on the bots, but it's a problem when running locally. It also wasn't happening when I landed the test, so it's a regression since then, maybe caused by the WorkerPool migration.

-----------------------------------------------------

signal 11 (SIGSEGV), code 1, fault addr 0x18 in tid 24084 (Thread-2)
pid: 24067, tid: 24084, name: Thread-2  >>> org.chromium.native_test:test_process <<<
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x18
     r0 00000018  r1 91230634  r2 00000000  r3 007fffff
     r4 00000018  r5 00000018  r6 910731c0  r7 a7386170
     r8 90fa3008  r9 92bfe640  sl 92bfe680  fp 92bfe6d0
     ip 9480df00  sp 92bfe0b0  lr a73000b9  pc a72d2ca8

Stack Trace:
  RELADDR   FUNCTION                                                                 FILE:LINE
  00097ca8  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000c50b5  base::RunLoop::QuitWhenIdle()+12                                         /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  0048243f  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00482da1  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  001f49c9  net::DnsConfigService::OnConfigRead(net::DnsConfig const&)+324           /data/app/org.chromium.native_test-2/lib/arm/libnet.cr.so
  001f5947  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/libnet.cr.so
  00204e27  net::SerialWorker::OnWorkJobFinished()+82                                /data/app/org.chromium.native_test-2/lib/arm/libnet.cr.so
  0008c435  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000e0c5d  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  0008c435  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  0009564d  base::debug::TaskAnnotator::RunTask(char const*, base::PendingTask*)+80  /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000abb45  base::internal::IncomingTaskQueue::RunTask(base::PendingTask*)+60        /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000acd5f  base::MessageLoop::RunTask(base::PendingTask*)+174                       /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000acfb5  base::MessageLoop::DeferOrRunPendingTask(base::PendingTask)+40           /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000ad119  base::MessageLoop::DoWork()+148                                          /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000af075  base::MessagePumpLibevent::Run(base::MessagePump::Delegate*)+44          /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000acb6f  base::MessageLoop::Run()+58                                              /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000c4d73  base::RunLoop::Run()+70                                                  /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  000c4fdf  base::RunLoop::RunUntilIdle()+54                                         /data/app/org.chromium.native_test-2/lib/arm/libbase.cr.so
  00482309  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00c550e5  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00c55521  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00c5573f  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00c5811d  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00c57f73  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00ca6f5f  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00b0a5f9  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00cb11a5  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00b0a573  <unknown>                                                                /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  00c611d1  Java_org_chromium_native_1test_NativeTest_nativeRunTests+440             /data/app/org.chromium.native_test-2/lib/arm/lib_net_unittests__library.cr.so
  003f0951  offset 0x3cf000                                                          /data/app/org.chromium.native_test-2/oat/arm/base.odex
 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 17 2017

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

commit 315bc9195bd21ce85cae46e7f52de767a04d5d48
Author: Miriam Gershenson <mgersh@chromium.org>
Date: Tue Oct 17 16:38:54 2017

Clean up DnsConfigServicePosixTest after TaskScheduler migration

The change caused this test to start crashing when the device has a
network connection. It also made some APIs for flushing tasks available,
so while I'm here, switch to using those.

Bug:  771627 
Change-Id: I2a031ceaceb0fc613532fcb65aadafd2a605034d
Reviewed-on: https://chromium-review.googlesource.com/721070
Commit-Queue: Miriam Gershenson <mgersh@chromium.org>
Reviewed-by: Paul Jensen <pauljensen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509408}
[modify] https://crrev.com/315bc9195bd21ce85cae46e7f52de767a04d5d48/net/dns/dns_config_service_posix_unittest.cc

Comment 2 by mge...@chromium.org, Oct 17 2017

Status: Fixed (was: Assigned)

Sign in to add a comment