[gem5-dev] changeset in gem5: Ruby System, Cache Recorder: Use delete [] fo...

[Joel Hestness]

changeset fa31189e1fb5 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=fa31189e1fb5
description:
        Ruby System, Cache Recorder: Use delete [] for trace vars

        The cache trace variables are array allocated uint8_t* in the 
RubySystem and
        the Ruby CacheRecorder, but the code used delete to free the memory, 
resulting
        in Valgrind memory errors. Change these deletes to delete [] to get rid 
of the
        errors.

diffstat:

 src/mem/ruby/recorder/CacheRecorder.cc |  2 +-
 src/mem/ruby/system/System.cc          |  2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diffs (24 lines):

diff -r 3b51f4defa0d -r fa31189e1fb5 src/mem/ruby/recorder/CacheRecorder.cc
--- a/src/mem/ruby/recorder/CacheRecorder.cc    Tue Apr 02 12:46:49 2013 -0400
+++ b/src/mem/ruby/recorder/CacheRecorder.cc    Sun Apr 07 20:31:15 2013 -0500
@@ -61,7 +61,7 @@
 CacheRecorder::~CacheRecorder()
 {
     if (m_uncompressed_trace != NULL) {
-        delete m_uncompressed_trace;
+        delete [] m_uncompressed_trace;
         m_uncompressed_trace = NULL;
     }
     m_seq_map.clear();
diff -r 3b51f4defa0d -r fa31189e1fb5 src/mem/ruby/system/System.cc
--- a/src/mem/ruby/system/System.cc     Tue Apr 02 12:46:49 2013 -0400
+++ b/src/mem/ruby/system/System.cc     Sun Apr 07 20:31:15 2013 -0500
@@ -331,7 +331,7 @@
                             memory_trace_size);
         m_mem_vec_ptr->populatePages(uncompressed_trace);
 
-        delete uncompressed_trace;
+        delete [] uncompressed_trace;
         uncompressed_trace = NULL;
     }
 
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev