From 2609ed2a7cf736834cafcf0b07465c45dfb567e3 Mon Sep 17 00:00:00 2001
From: Ali Saidi <saidi@eecs.umich.edu>
Date: Fri, 7 Apr 2006 16:26:22 -0400
Subject: [PATCH] a bit of bad code trampling on memory

--HG--
extra : convert_revision : c0252dce6d7fc4c35ecd9f87ac4555e704de91b7
---
 mem/bus.cc      |  4 +++-
 mem/bus.hh      | 13 +++++--------
 mem/physical.cc |  2 +-
 3 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/mem/bus.cc b/mem/bus.cc
index 0cadc2045..8e8bc2203 100644
--- a/mem/bus.cc
+++ b/mem/bus.cc
@@ -85,8 +85,10 @@ Bus::recvFunctional(Packet &pkt, int id)
 void
 Bus::recvStatusChange(Port::Status status, int id)
 {
-    assert(status == Port:: RangeChange &&
+    assert(status == Port::RangeChange &&
            "The other statuses need to be implemented.");
+
+    assert(id < interfaces.size() && id >= 0);
     Port *port = interfaces[id];
     AddrRangeList ranges;
     AddrRangeList snoops;
diff --git a/mem/bus.hh b/mem/bus.hh
index eff42c55a..fad44aba5 100644
--- a/mem/bus.hh
+++ b/mem/bus.hh
@@ -127,12 +127,9 @@ class Bus : public MemObject
 
     };
 
-    /** A count of the number of interfaces connected to this bus. */
-    int num_interfaces;
-
     /** An array of pointers to the peer port interfaces
         connected to this bus.*/
-    Port *interfaces[];
+    std::vector<Port*> interfaces;
 
   public:
 
@@ -140,12 +137,12 @@ class Bus : public MemObject
     virtual Port *getPort(const std::string &if_name)
     {
         // if_name ignored?  forced to be empty?
-        int id = num_interfaces++;
-        interfaces[id] = new BusPort(this, id);
-        return interfaces[id];
+        int id = interfaces.size();
+        interfaces.push_back(new BusPort(this, id));
+        return interfaces.back();
     }
     Bus(const std::string &n)
-        : MemObject(n), num_interfaces(0) {}
+        : MemObject(n)  {}
 
 };
 
diff --git a/mem/physical.cc b/mem/physical.cc
index 4087f3e32..603f8f63e 100644
--- a/mem/physical.cc
+++ b/mem/physical.cc
@@ -70,7 +70,7 @@ PhysicalMemory::MemResponseEvent::description()
 }
 
 PhysicalMemory::PhysicalMemory(const string &n)
-    : MemObject(n), base_addr(0), pmem_addr(NULL)
+    : MemObject(n), base_addr(0), pmem_addr(NULL), port(NULL)
 {
     // Hardcoded to 128 MB for now.
     pmem_size = 1 << 27;
-- 
2.30.2