summaryrefslogtreecommitdiff
path: root/src/mem/simple_mem.cc
diff options
context:
space:
mode:
authorAli Saidi <Ali.Saidi@ARM.com>2012-11-02 11:50:16 -0500
committerAli Saidi <Ali.Saidi@ARM.com>2012-11-02 11:50:16 -0500
commitce5766c409ca3e27e62a300b89511b9d0fcd6c18 (patch)
tree32ef02e398c9edec9ee0acb926d6fd925b3d069f /src/mem/simple_mem.cc
parent1dbf9bb4ca6cc3bee68713a28778c1bdfe222f75 (diff)
downloadgem5-ce5766c409ca3e27e62a300b89511b9d0fcd6c18.tar.xz
mem: fix use after free issue in memories until 4-phase work complete.
Diffstat (limited to 'src/mem/simple_mem.cc')
-rw-r--r--src/mem/simple_mem.cc8
1 files changed, 7 insertions, 1 deletions
diff --git a/src/mem/simple_mem.cc b/src/mem/simple_mem.cc
index e78b57928..d3a53a26f 100644
--- a/src/mem/simple_mem.cc
+++ b/src/mem/simple_mem.cc
@@ -94,10 +94,16 @@ SimpleMemory::doFunctionalAccess(PacketPtr pkt)
bool
SimpleMemory::recvTimingReq(PacketPtr pkt)
{
+ /// @todo temporary hack to deal with memory corruption issues until
+ /// 4-phase transactions are complete
+ for (int x = 0; x < pendingDelete.size(); x++)
+ delete pendingDelete[x];
+ pendingDelete.clear();
+
if (pkt->memInhibitAsserted()) {
// snooper will supply based on copy of packet
// still target's responsibility to delete packet
- delete pkt;
+ pendingDelete.push_back(pkt);
return true;
}