diff options
author | Benjamin Kramer <benny.kra@googlemail.com> | 2011-02-24 11:03:19 +0000 |
---|---|---|
committer | Benjamin Kramer <benny.kra@googlemail.com> | 2011-02-24 11:03:19 +0000 |
commit | c053644217dd35d492701d290052130e1d76c0df (patch) | |
tree | d10c02b8fece631fef2d5997244903c70da1bd1c /llvm/tools/llvm-mc/Disassembler.cpp | |
parent | a62fc89a04b2fb9d15a7c14332bdfcfb59adc129 (diff) | |
download | bcm5719-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
Diffstat (limited to 'llvm/tools/llvm-mc/Disassembler.cpp')
-rw-r--r-- | llvm/tools/llvm-mc/Disassembler.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
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; |