From 5bdeb1a432430dcb265cef77fa9365eceddce7d7 Mon Sep 17 00:00:00 2001
From: Gabe Black <gabeblack@google.com>
Date: Thu, 27 Sep 2018 01:16:42 -0700
Subject: [PATCH] systemc: Modify reportifyException to keep sc_reports in
 scope.

reportifyException was sometimes indirectly creating temporary
sc_report objects which would go out of scope when they were
returned. The later code which tried to print them would then read
garbage.

Change-Id: I0a744eb029c62bf2ffee83db0a0e6dcbe3e60f7d
Reviewed-on: https://gem5-review.googlesource.com/c/13190
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>
---
 src/systemc/core/sc_main.cc   |  3 +--
 src/systemc/core/scheduler.cc | 11 +++++------
 src/systemc/core/scheduler.hh |  4 ++--
 3 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/src/systemc/core/sc_main.cc b/src/systemc/core/sc_main.cc
index e9df6a1e0..59ec9f8bd 100644
--- a/src/systemc/core/sc_main.cc
+++ b/src/systemc/core/sc_main.cc
@@ -81,8 +81,7 @@ class ScMainFiber : public Fiber
                 resultStr = r.what();
             } catch (...) {
                 // There was some other type of exception we need to wrap.
-                const sc_report *r = ::sc_gem5::reportifyException();
-                resultStr = r->what();
+                resultStr = ::sc_gem5::reportifyException().what();
             }
             ::sc_gem5::Kernel::scMainFinished(true);
             ::sc_gem5::scheduler.clear();
diff --git a/src/systemc/core/scheduler.cc b/src/systemc/core/scheduler.cc
index fe1d7c2e4..d57505517 100644
--- a/src/systemc/core/scheduler.cc
+++ b/src/systemc/core/scheduler.cc
@@ -418,11 +418,10 @@ Scheduler::schedulePause()
 }
 
 void
-Scheduler::throwToScMain(const ::sc_core::sc_report *r)
+Scheduler::throwToScMain()
 {
-    if (!r)
-        r = reportifyException();
-    _throwToScMain = r;
+    ::sc_core::sc_report report = reportifyException();
+    _throwToScMain = &report;
     status(StatusOther);
     scMain->run();
 }
@@ -462,7 +461,7 @@ throwingReportHandler(const ::sc_core::sc_report &r,
 
 } // anonymous namespace
 
-const ::sc_core::sc_report *
+const ::sc_core::sc_report
 reportifyException()
 {
     ::sc_core::sc_report_handler_proc old_handler =
@@ -488,7 +487,7 @@ reportifyException()
         }
     } catch (const ::sc_core::sc_report &r) {
         ::sc_core::sc_report_handler::set_handler(old_handler);
-        return &r;
+        return r;
     }
     panic("No exception thrown in reportifyException.");
 }
diff --git a/src/systemc/core/scheduler.hh b/src/systemc/core/scheduler.hh
index de9d627c1..83585dd2d 100644
--- a/src/systemc/core/scheduler.hh
+++ b/src/systemc/core/scheduler.hh
@@ -365,7 +365,7 @@ class Scheduler
 
     uint64_t changeStamp() { return _changeStamp; }
 
-    void throwToScMain(const ::sc_core::sc_report *r=nullptr);
+    void throwToScMain();
 
     Status status() { return _status; }
     void status(Status s) { _status = s; }
@@ -511,7 +511,7 @@ Scheduler::TimeSlot::process()
     scheduler.completeTimeSlot(this);
 }
 
-const ::sc_core::sc_report *reportifyException();
+const ::sc_core::sc_report reportifyException();
 
 } // namespace sc_gem5
 
-- 
2.30.2