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

Chris Lattner lattner at cs.uiuc.edu
Fri May 12 11:10:24 PDT 2006



Changes in directory llvm/lib/ExecutionEngine/JIT:

JITEmitter.cpp updated: 1.100 -> 1.101
---
Log message:

Fix a hypothetical memory leak, identified by Coverity.  In practice, this
object is never deleted though.


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

 JITEmitter.cpp |    4 +++-
 1 files changed, 3 insertions(+), 1 deletion(-)


Index: llvm/lib/ExecutionEngine/JIT/JITEmitter.cpp
diff -u llvm/lib/ExecutionEngine/JIT/JITEmitter.cpp:1.100 llvm/lib/ExecutionEngine/JIT/JITEmitter.cpp:1.101
--- llvm/lib/ExecutionEngine/JIT/JITEmitter.cpp:1.100	Thu May 11 19:03:12 2006
+++ llvm/lib/ExecutionEngine/JIT/JITEmitter.cpp	Fri May 12 13:10:12 2006
@@ -392,12 +392,14 @@
 
   // Allocate the GOT.
   GOTBase = NULL;
-  if (useGOT) GOTBase = (unsigned char*)malloc(sizeof(void*) * 8192);
+  if (useGOT) GOTBase = new unsigned char[sizeof(void*) * 8192];
 }
 
 JITMemoryManager::~JITMemoryManager() {
   for (unsigned i = 0, e = Blocks.size(); i != e; ++i)
     sys::Memory::ReleaseRWX(Blocks[i]);
+  
+  delete[] GOTBase;
   Blocks.clear();
 }
 






More information about the llvm-commits mailing list