diff options
author | Christopher Ferris <cferris@google.com> | 2015-04-27 15:19:41 -0700 |
---|---|---|
committer | Christopher Ferris <cferris@google.com> | 2015-05-04 12:10:07 -0700 |
commit | 241a9588c6d7d0fcb4c92da40c7141863930083a (patch) | |
tree | 4a6868893dc742f6322775d198a2305f645f2f3f | |
parent | a93b104b47da67eaa50f8a6b7280cc6c438ef37b (diff) | |
download | art-241a9588c6d7d0fcb4c92da40c7141863930083a.zip art-241a9588c6d7d0fcb4c92da40c7141863930083a.tar.gz art-241a9588c6d7d0fcb4c92da40c7141863930083a.tar.bz2 |
Fix mismatched new[]/delete.
Another two cases where a new[] is used but only a delete occurs.
Bug: 18202869
Change-Id: If68264807150f3a9783e44ef8823cc366bff8df2
-rw-r--r-- | runtime/interpreter/interpreter.cc | 2 | ||||
-rw-r--r-- | runtime/quick_exception_handler.cc | 3 | ||||
-rw-r--r-- | runtime/stack.h | 10 | ||||
-rw-r--r-- | runtime/trace.h | 2 |
4 files changed, 12 insertions, 5 deletions
diff --git a/runtime/interpreter/interpreter.cc b/runtime/interpreter/interpreter.cc index 423b952..a37aee5 100644 --- a/runtime/interpreter/interpreter.cc +++ b/runtime/interpreter/interpreter.cc @@ -423,7 +423,7 @@ void EnterInterpreterFromDeoptimize(Thread* self, ShadowFrame* shadow_frame, JVa } ShadowFrame* old_frame = shadow_frame; shadow_frame = shadow_frame->GetLink(); - delete old_frame; + ShadowFrame::DeleteDeoptimizedFrame(old_frame); } ret_val->SetJ(value.GetJ()); } diff --git a/runtime/quick_exception_handler.cc b/runtime/quick_exception_handler.cc index 2432603..a80eed6 100644 --- a/runtime/quick_exception_handler.cc +++ b/runtime/quick_exception_handler.cc @@ -202,7 +202,8 @@ class DeoptimizeStackVisitor FINAL : public StackVisitor { h_method, m->GetAccessFlags(), true, true, true, true); bool verifier_success = verifier.Verify(); CHECK(verifier_success) << PrettyMethod(h_method.Get()); - ShadowFrame* new_frame = ShadowFrame::Create(num_regs, nullptr, h_method.Get(), dex_pc); + ShadowFrame* new_frame = ShadowFrame::CreateDeoptimizedFrame( + num_regs, nullptr, h_method.Get(), dex_pc); self_->SetShadowFrameUnderConstruction(new_frame); const std::vector<int32_t> kinds(verifier.DescribeVRegs(dex_pc)); diff --git a/runtime/stack.h b/runtime/stack.h index e2af5ee..3f1bff8 100644 --- a/runtime/stack.h +++ b/runtime/stack.h @@ -74,12 +74,18 @@ class ShadowFrame { } // Create ShadowFrame in heap for deoptimization. - static ShadowFrame* Create(uint32_t num_vregs, ShadowFrame* link, - mirror::ArtMethod* method, uint32_t dex_pc) { + static ShadowFrame* CreateDeoptimizedFrame(uint32_t num_vregs, ShadowFrame* link, + mirror::ArtMethod* method, uint32_t dex_pc) { uint8_t* memory = new uint8_t[ComputeSize(num_vregs)]; return Create(num_vregs, link, method, dex_pc, memory); } + // Delete a ShadowFrame allocated on the heap for deoptimization. + static void DeleteDeoptimizedFrame(ShadowFrame* sf) { + uint8_t* memory = reinterpret_cast<uint8_t*>(sf); + delete[] memory; + } + // Create ShadowFrame for interpreter using provided memory. static ShadowFrame* Create(uint32_t num_vregs, ShadowFrame* link, mirror::ArtMethod* method, uint32_t dex_pc, void* memory) { diff --git a/runtime/trace.h b/runtime/trace.h index 06824b8..df6d5e7 100644 --- a/runtime/trace.h +++ b/runtime/trace.h @@ -189,7 +189,7 @@ class Trace FINAL : public instrumentation::InstrumentationListener { std::unique_ptr<File> trace_file_; // Buffer to store trace data. - std::unique_ptr<uint8_t> buf_; + std::unique_ptr<uint8_t[]> buf_; // Flags enabling extra tracing of things such as alloc counts. const int flags_; |