From ed15000a5099f5e230c8ded5ac75692bae272650 Mon Sep 17 00:00:00 2001 From: Mathieu Chartier Date: Fri, 28 Aug 2015 11:16:54 -0700 Subject: Fix some HandleScope bugs and add corresponding checks Some places were creating or destroying handle scopes without holding the mutator lock. This can cause GC crashes if thread roots are being marked or hprof dumps to also fail. Also added checks to catch some of these errors. Bug: 23468617 Change-Id: I1a2d615923484cfc25014967656775c445aa3f1f --- compiler/driver/compiler_driver_test.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'compiler/driver/compiler_driver_test.cc') diff --git a/compiler/driver/compiler_driver_test.cc b/compiler/driver/compiler_driver_test.cc index e35d07da83..a5ace0bf42 100644 --- a/compiler/driver/compiler_driver_test.cc +++ b/compiler/driver/compiler_driver_test.cc @@ -210,8 +210,8 @@ TEST_F(CompilerDriverMethodsTest, Selection) { CompileAll(class_loader); ClassLinker* class_linker = Runtime::Current()->GetClassLinker(); - StackHandleScope<1> hs(self); ScopedObjectAccess soa(self); + StackHandleScope<1> hs(self); Handle h_loader(hs.NewHandle( reinterpret_cast(self->DecodeJObject(class_loader)))); mirror::Class* klass = class_linker->FindClass(self, "LStaticLeafMethods;", h_loader); -- cgit v1.2.3-59-g8ed1b