diff options
| author | Benedikt Waibel <benedikt.waibel@student.kit.edu> | 2025-08-12 20:51:54 +0200 |
|---|---|---|
| committer | Brandon Miller <bkmiller89@icloud.com> | 2025-08-18 11:45:10 -0400 |
| commit | f898679226e1051695c45758047fa853acb68532 (patch) | |
| tree | b00ab5e37883cd912b3871478c418ac853b4ec7c /defaultabb.cpp | |
| parent | fe064e59aabb8ee362c587a04f081f77619f0313 (diff) | |
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.
Diffstat (limited to 'defaultabb.cpp')
| -rw-r--r-- | defaultabb.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
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", |