sanchayanmaity/gem5
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Ruby System, Cache Recorder: Use delete [] for trace vars

Browse source 
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.
This commit is contained in:
Joel Hestness 2013-04-07 20:31:15 -05:00
parent 7fb55b98cc
commit 3b02210713
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/mem/ruby/recorder/CacheRecorder.cc
View file

@ -61,7 +61,7 @@ CacheRecorder::CacheRecorder(uint8_t* uncompressed_trace,
CacheRecorder::~CacheRecorder() CacheRecorder::~CacheRecorder()
{ {
if (m_uncompressed_trace != NULL) { if (m_uncompressed_trace != NULL) {
delete m_uncompressed_trace; delete [] m_uncompressed_trace;
m_uncompressed_trace = NULL; m_uncompressed_trace = NULL;
} }
m_seq_map.clear(); m_seq_map.clear();

2
src/mem/ruby/system/System.cc
View file

@ -331,7 +331,7 @@ RubySystem::unserialize(Checkpoint *cp, const string &section)
memory_trace_size); memory_trace_size);
m_mem_vec_ptr->populatePages(uncompressed_trace); m_mem_vec_ptr->populatePages(uncompressed_trace);
delete uncompressed_trace; delete [] uncompressed_trace;
uncompressed_trace = NULL; uncompressed_trace = NULL;
} }