Make sure that output files are always checked success before they're used.

Make OutputDirectory::resolve() private and change the functions using
resolve() to instead use create().

--HG--
extra : convert_revision : 36d4be629764d0c4c708cec8aa712cd15f966453

1 files changed, 2 insertions, 3 deletions

diff --git a/src/mem/cache/tags/split.cc b/src/mem/cache/tags/split.cc
index 0df85cc92..5bed44bd3 100644
--- a/src/mem/cache/tags/split.cc
+++ b/src/mem/cache/tags/split.cc
@@ -379,13 +379,12 @@ Split::cleanupRefs()
     else if (lru_net)
         lru_net->cleanupRefs();
 
-    ofstream memPrint(simout.resolve("memory_footprint.txt").c_str(),
-                      ios::trunc);
+    ostream *memPrint = simout.create("memory_footprint.txt");
 
     // this shouldn't be here but it happens at the end, which is what i want
     memIter end = memHash.end();
     for (memIter iter = memHash.begin(); iter != end; ++iter) {
-        ccprintf(memPrint, "%8x\t%d\n", (*iter).first, (*iter).second);
+        ccprintf(*memPrint, "%8x\t%d\n", (*iter).first, (*iter).second);
     }
 }