From: Benjamin Kramer Date: Thu, 24 Mar 2011 21:14:28 +0000 (+0000) Subject: Plug a leak in the arm disassembler and put the tests back. X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=83ccbff84fd38d8680ae39b3b629aee339478855;p=oota-llvm.git Plug a leak in the arm disassembler and put the tests back. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@128238 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Target/ARM/Disassembler/ARMDisassembler.cpp b/lib/Target/ARM/Disassembler/ARMDisassembler.cpp index f7afac64af3..325cffb0fe4 100644 --- a/lib/Target/ARM/Disassembler/ARMDisassembler.cpp +++ b/lib/Target/ARM/Disassembler/ARMDisassembler.cpp @@ -18,6 +18,7 @@ #include "ARMDisassembler.h" #include "ARMDisassemblerCore.h" +#include "llvm/ADT/OwningPtr.h" #include "llvm/MC/EDInstInfo.h" #include "llvm/MC/MCInst.h" #include "llvm/Target/TargetRegistry.h" @@ -384,15 +385,13 @@ bool ARMDisassembler::getInstruction(MCInst &MI, showBitVector(errs(), insn); }); - ARMBasicMCBuilder *Builder = CreateMCBuilder(Opcode, Format); + OwningPtr Builder(CreateMCBuilder(Opcode, Format)); if (!Builder) return false; if (!Builder->Build(MI, insn)) return false; - delete Builder; - return true; } diff --git a/test/MC/Disassembler/ARM/invalid-CPS3p-arm.txt b/test/MC/Disassembler/ARM/invalid-CPS3p-arm.txt new file mode 100644 index 00000000000..5202217b6a7 --- /dev/null +++ b/test/MC/Disassembler/ARM/invalid-CPS3p-arm.txt @@ -0,0 +1,4 @@ +# RUN: llvm-mc --disassemble %s -triple=arm-apple-darwin9 |& grep {invalid instruction encoding} + +# invalid (imod, M, iflags) combination +0x93 0x1c 0x02 0xf1 diff --git a/test/MC/Disassembler/ARM/invalid-VLDMSDB-arm.txt b/test/MC/Disassembler/ARM/invalid-VLDMSDB-arm.txt new file mode 100644 index 00000000000..f57ddbcafd6 --- /dev/null +++ b/test/MC/Disassembler/ARM/invalid-VLDMSDB-arm.txt @@ -0,0 +1,4 @@ +# RUN: llvm-mc --disassemble %s -triple=arm-apple-darwin9 |& grep {invalid instruction encoding} + +# core registers out of range +0xa5 0xba 0x52 0xed