summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBenjamin Kramer <benny.kra@googlemail.com>2011-02-24 11:03:19 +0000
committerBenjamin Kramer <benny.kra@googlemail.com>2011-02-24 11:03:19 +0000
commitc053644217dd35d492701d290052130e1d76c0df (patch)
treed10c02b8fece631fef2d5997244903c70da1bd1c
parenta62fc89a04b2fb9d15a7c14332bdfcfb59adc129 (diff)
downloadbcm5719-llvm-c053644217dd35d492701d290052130e1d76c0df.tar.gz
bcm5719-llvm-c053644217dd35d492701d290052130e1d76c0df.zip
Plug some leaks in edis.
- Don't leak parsed operands during tokenization. - Don't leak printed insts in llvm-mc. llvm-svn: 126381
-rw-r--r--llvm/lib/MC/MCDisassembler/EDToken.cpp4
-rw-r--r--llvm/tools/llvm-mc/Disassembler.cpp10
2 files changed, 9 insertions, 5 deletions
diff --git a/llvm/lib/MC/MCDisassembler/EDToken.cpp b/llvm/lib/MC/MCDisassembler/EDToken.cpp
index 400e1649e97..de770b41ef3 100644
--- a/llvm/lib/MC/MCDisassembler/EDToken.cpp
+++ b/llvm/lib/MC/MCDisassembler/EDToken.cpp
@@ -194,6 +194,10 @@ int EDToken::tokenize(std::vector<EDToken*> &tokens,
tokens.push_back(token);
}
+ // Free any parsed operands.
+ for (unsigned i = 0, e = parsedOperands.size(); i != e; ++i)
+ delete parsedOperands[i];
+
return 0;
}
diff --git a/llvm/tools/llvm-mc/Disassembler.cpp b/llvm/tools/llvm-mc/Disassembler.cpp
index 1f0850ff8b4..d98b57ebc65 100644
--- a/llvm/tools/llvm-mc/Disassembler.cpp
+++ b/llvm/tools/llvm-mc/Disassembler.cpp
@@ -227,8 +227,8 @@ int Disassembler::disassembleEnhanced(const std::string &TS,
}
EDDisassembler::initialize();
- EDDisassembler *disassembler =
- EDDisassembler::getDisassembler(TS.c_str(), AS);
+ OwningPtr<EDDisassembler>
+ disassembler(EDDisassembler::getDisassembler(TS.c_str(), AS));
if (disassembler == 0) {
errs() << "error: couldn't get disassembler for " << TS << '\n';
@@ -236,8 +236,8 @@ int Disassembler::disassembleEnhanced(const std::string &TS,
}
while (ByteArray.size()) {
- EDInst *inst =
- disassembler->createInst(byteArrayReader, 0, &ByteArray);
+ OwningPtr<EDInst>
+ inst(disassembler->createInst(byteArrayReader, 0, &ByteArray));
ByteArray.erase (ByteArray.begin(), ByteArray.begin() + inst->byteSize());
@@ -330,7 +330,7 @@ int Disassembler::disassembleEnhanced(const std::string &TS,
}
uint64_t evaluatedResult;
- void *Arg[] = { disassembler, &Out };
+ void *Arg[] = { disassembler.get(), &Out };
if (operand->evaluate(evaluatedResult, verboseEvaluator, Arg)) {
errs() << "error: Couldn't evaluate an operand\n";
return -1;
OpenPOWER on IntegriCloud