commit e7a5b7d3fcc3af100fec13af057eecaff1037f2c
author Ian Rogers <irogers@google.com> Thu Apr 18 20:09:02 2013 -0700
committer Ian Rogers <irogers@google.com> Thu Apr 18 20:09:02 2013 -0700
tree 17c679d40aaaea7ddb53fdc61494c8d243af9ce5
parent 1bd4b4ca7f4f44c55ded050e5a6be09811e1b283

Fix memory leak of and use without initialization.

Arena allocator's empty arena needs to be released in a destructor, this
removes the neeed for Reset.
The DataflowIterator needs to clear changed_ upon construction to avoid use
without initialization.

Change-Id: I56e3cb8c3e06c08ab0ff42447bd6e05792fc70af

src/compiler/dex/dataflow_iterator.h

diff --git a/src/compiler/dex/dataflow_iterator.h b/src/compiler/dex/dataflow_iterator.h
index 7d32dfc..a4b38bd 100644
--- a/src/compiler/dex/dataflow_iterator.h
+++ b/src/compiler/dex/dataflow_iterator.h
@@ -66,7 +66,10 @@
             is_iterative_(is_iterative),
             start_idx_(start_idx),
             end_idx_(end_idx),
-            reverse_(reverse) {}
+            reverse_(reverse),
+            block_id_list_(NULL),
+            idx_(0),
+            changed_(false) {}
 
       virtual BasicBlock* NextBody(bool had_change);