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 | |
| 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.
| -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", |