changeset 8adc048515b8 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=8adc048515b8
description:
        EventManager: Remove test for NULL pointer in constructor

        This patch tidies up the EventManager constructor and prunes a corner
        case where the EventManager would initialise its eventq pointer to
        NULL. This would cause segmentation faults on actual use and should
        never happen.

diffstat:

 src/sim/eventq.hh |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff -r d152d34a4adf -r 8adc048515b8 src/sim/eventq.hh
--- a/src/sim/eventq.hh Tue Aug 21 05:49:09 2012 -0400
+++ b/src/sim/eventq.hh Tue Aug 21 05:49:18 2012 -0400
@@ -432,7 +432,7 @@
 
   public:
     EventManager(EventManager &em) : eventq(em.eventq) {}
-    EventManager(EventManager *em) : eventq(em ? em->eventq : NULL) {}
+    EventManager(EventManager *em) : eventq(em->eventq) {}
     EventManager(EventQueue *eq) : eventq(eq) {}
 
     EventQueue *
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to