diff options
| author | Andreas Sandberg <Andreas.Sandberg@ARM.com> | 2014-09-09 04:36:32 -0400 |
|---|---|---|
| committer | Andreas Sandberg <Andreas.Sandberg@ARM.com> | 2014-09-09 04:36:32 -0400 |
| commit | 11494c4345e0de2b51dddcc936cedade76f8a90e (patch) | |
| tree | a09f4fbe426871acea19d5171531f682dd06254a /src/sim/global_event.cc | |
| parent | da4539dc749c3d29c03de9b3130f1c9a7266be9d (diff) | |
| download | gem5-11494c4345e0de2b51dddcc936cedade76f8a90e.tar.xz | |
sim: Fix resource leak in BaseGlobalEvent
Static analysis revealed that BaseGlobalEvent::barrier was never
deallocated. This changeset solves this leak by making the barrier
allocation a part of the BaseGlobalEvent instead of storing a pointer
to a separate heap-allocated barrier.
Diffstat (limited to 'src/sim/global_event.cc')
| -rw-r--r-- | src/sim/global_event.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/sim/global_event.cc b/src/sim/global_event.cc index fedee351f..f01df51d0 100644 --- a/src/sim/global_event.cc +++ b/src/sim/global_event.cc @@ -34,9 +34,9 @@ std::mutex BaseGlobalEvent::globalQMutex; BaseGlobalEvent::BaseGlobalEvent(Priority p, Flags f) + : barrier(numMainEventQueues), + barrierEvent(numMainEventQueues, NULL) { - barrierEvent.resize(numMainEventQueues); - barrier = new Barrier(numMainEventQueues); } |