Merge branch 'dev' into devtooligan-suggestions/arbitrary-from-strategy

Cyfrin · Oct 19, 2024 · a0d87df · a0d87df
2 parents 5719621 + f855512
commit a0d87df
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/aderyn_core/src/detect/low/unsafe_erc20_functions.rs b/aderyn_core/src/detect/low/unsafe_erc20_functions.rs
@@ -18,7 +18,9 @@ pub struct UnsafeERC20FunctionsDetector {
 impl IssueDetector for UnsafeERC20FunctionsDetector {
     fn detect(&mut self, context: &WorkspaceContext) -> Result<bool, Box<dyn Error>> {
         for member_access in context.member_accesses() {
-            if member_access.member_name == "transferFrom"
+            if member_access.expression.as_ref().type_descriptions().is_some_and(|desc| {
+                desc.type_string.as_ref().is_some_and(|type_string| type_string.contains("ERC20"))
+            }) && member_access.member_name == "transferFrom"
                 || member_access.member_name == "approve"
                 || member_access.member_name == "transfer"
             {

diff --git a/reports/templegold-report.md b/reports/templegold-report.md