[llvm-commits] CVS: llvm/lib/ExecutionEngine/ExecutionEngine.cpp

Chris Lattner lattner at cs.uiuc.edu
Sun May 14 12:02:07 PDT 2006



Changes in directory llvm/lib/ExecutionEngine:

ExecutionEngine.cpp updated: 1.82 -> 1.83
---
Log message:

LoadLibraryPermanently can theoretically throw an exception.  Do not propagate
it out of 'ExecutionEngine::create'.  This fixes a problem reported by coverity.


---
Diffs of the changes:  (+4 -1)

 ExecutionEngine.cpp |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletion(-)


Index: llvm/lib/ExecutionEngine/ExecutionEngine.cpp
diff -u llvm/lib/ExecutionEngine/ExecutionEngine.cpp:1.82 llvm/lib/ExecutionEngine/ExecutionEngine.cpp:1.83
--- llvm/lib/ExecutionEngine/ExecutionEngine.cpp:1.82	Mon May  8 17:00:52 2006
+++ llvm/lib/ExecutionEngine/ExecutionEngine.cpp	Sun May 14 14:01:55 2006
@@ -247,7 +247,10 @@
   if (EE) {
     // Make sure we can resolve symbols in the program as well. The zero arg
     // to the function tells DynamicLibrary to load the program, not a library.
-    sys::DynamicLibrary::LoadLibraryPermanently(0);
+    try {
+      sys::DynamicLibrary::LoadLibraryPermanently(0);
+    } catch (...) {
+    }
   }
 
   return EE;






More information about the llvm-commits mailing list