From f898679226e1051695c45758047fa853acb68532 Mon Sep 17 00:00:00 2001 From: Benedikt Waibel Date: Tue, 12 Aug 2025 20:51:54 +0200 Subject: Fix disallowBranchToString in default abb. The setting should halt analysis of branch targets that fall within string references. Previous implementation looked up string references at branch source. --- defaultabb.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'defaultabb.cpp') diff --git a/defaultabb.cpp b/defaultabb.cpp index 647bbfb6..6c3fc270 100644 --- a/defaultabb.cpp +++ b/defaultabb.cpp @@ -462,7 +462,7 @@ void Architecture::DefaultAnalyzeBasicBlocks(Function* function, BasicBlockAnaly break; } } - else if (disallowBranchToString && data->GetStringAtAddress(location.address, strRef) && targetExceedsByteLimit(strRef)) + else if (disallowBranchToString && data->GetStringAtAddress(target.address, strRef) && targetExceedsByteLimit(strRef)) { BNLogInfo("Not adding branch target from 0x%" PRIx64 " to string at 0x%" PRIx64 " length:%zu", -- cgit v1.3.1