diff options
author | Joel Hestness <jthestness@gmail.com> | 2013-04-07 20:31:15 -0500 |
---|---|---|
committer | Joel Hestness <jthestness@gmail.com> | 2013-04-07 20:31:15 -0500 |
commit | 3b02210713a3b8f46750638003784115a738ec78 (patch) | |
tree | d3787d1380716d3fdb03fc4c98b6bc8ebe3ac699 /src/mem | |
parent | 7fb55b98cc63e8c08a198ae8bc9396ae96bfabe5 (diff) | |
download | gem5-3b02210713a3b8f46750638003784115a738ec78.tar.xz |
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 (limited to 'src/mem')
-rw-r--r-- | src/mem/ruby/recorder/CacheRecorder.cc | 2 | ||||
-rw-r--r-- | src/mem/ruby/system/System.cc | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/mem/ruby/recorder/CacheRecorder.cc b/src/mem/ruby/recorder/CacheRecorder.cc index c5593d945..1e029b400 100644 --- a/src/mem/ruby/recorder/CacheRecorder.cc +++ b/src/mem/ruby/recorder/CacheRecorder.cc @@ -61,7 +61,7 @@ CacheRecorder::CacheRecorder(uint8_t* uncompressed_trace, CacheRecorder::~CacheRecorder() { if (m_uncompressed_trace != NULL) { - delete m_uncompressed_trace; + delete [] m_uncompressed_trace; m_uncompressed_trace = NULL; } m_seq_map.clear(); diff --git a/src/mem/ruby/system/System.cc b/src/mem/ruby/system/System.cc index 0e52df2ec..617788b99 100644 --- a/src/mem/ruby/system/System.cc +++ b/src/mem/ruby/system/System.cc @@ -331,7 +331,7 @@ RubySystem::unserialize(Checkpoint *cp, const string §ion) memory_trace_size); m_mem_vec_ptr->populatePages(uncompressed_trace); - delete uncompressed_trace; + delete [] uncompressed_trace; uncompressed_trace = NULL; } |