Magic-Mirror/capstone
1
0
Fork 0
mirror of https://github.com/capstone-engine/capstone.git synced 2025-02-10 16:42:54 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix Thumb disassembler memory corruption with IT sequence (issue #385)

Browse Source
This commit is contained in:
Nikolay Igotti 2015-06-03 15:38:45 +02:00 committed by Nguyen Anh Quynh
parent 76c0c45ec0
commit 1d9615de78
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
arch/ARM/ARMDisassembler.c
View File

@ -44,6 +44,10 @@
static bool ITStatus_push_back(ARM_ITStatus *it, char v)
{
if (it->size >= sizeof(it->ITStates)) {
// TODO: consider warning user.
it->size = 0;
}
it->ITStates[it->size] = v;
it->size++;
@ -751,8 +755,7 @@ static DecodeStatus _Thumb_getInstruction(cs_struct *ud, MCInst *MI, const uint8
// Nested IT blocks are UNPREDICTABLE. Must be checked before we add
// the Thumb predicate.
if (MCInst_getOpcode(MI) == ARM_t2IT && ITStatus_instrInITBlock(&(ud->ITBlock)))
result = MCDisassembler_SoftFail;
return MCDisassembler_SoftFail;
Check(&result, AddThumbPredicate(ud, MI));
// If we find an IT instruction, we need to parse its condition