From a3a136411515a83a24cfd67e4c7210057ace4bd2 Mon Sep 17 00:00:00 2001
From: Chris Johns <chrisj@rtems.org>
Date: Tue, 21 Aug 2018 14:43:28 +1000
Subject: covoar: Close the ELF and DWARF handles once the data is loaded.

This avoids the processing being killed with to many file handles
open.
---
 tester/covoar/ExecutableInfo.cc | 25 +++++++++++++++++--------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/tester/covoar/ExecutableInfo.cc b/tester/covoar/ExecutableInfo.cc
index 0a629b7..d93860b 100644
--- a/tester/covoar/ExecutableInfo.cc
+++ b/tester/covoar/ExecutableInfo.cc
@@ -37,27 +37,36 @@ namespace Coverage {
     executable.open();
     executable.begin();
     executable.load_symbols(symbols);
+
     debug.begin(executable.elf());
     debug.load_debug();
     debug.load_functions();
 
-    for (auto& cu : debug.get_cus()) {
-      for (auto& func : cu.get_functions()) {
-        if (func.has_machine_code() && (!func.is_inlined() || func.is_external())) {
-          createCoverageMap (cu.name(), func.name(),
-                             func.pc_low(), func.pc_high());
+    try {
+      for (auto& cu : debug.get_cus()) {
+        for (auto& func : cu.get_functions()) {
+          if (func.has_machine_code() && (!func.is_inlined() || func.is_external())) {
+            createCoverageMap (cu.name(), func.name(),
+                               func.pc_low(), func.pc_high());
+          }
         }
       }
+    } catch (...) {
+      debug.end();
+      executable.end();
+      executable.close();
+      throw;
     }
-  }
 
-  ExecutableInfo::~ExecutableInfo()
-  {
     debug.end();
     executable.end();
     executable.close();
   }
 
+  ExecutableInfo::~ExecutableInfo()
+  {
+  }
+
   void ExecutableInfo::dumpCoverageMaps( void ) {
     ExecutableInfo::CoverageMaps::iterator  itr;
 
-- 
cgit v1.2.3