NFTBridge
60,000 USDC
View results
Submission Details
Severity: medium
Invalid

CollectionManager.sol - SWC-107: External calls in functions handling collection operations could be vulnerable., SWC-101: Operations on collection balances may cause overflows.

  • Summary: Analyzed for common Solidity vulnerabilities related to collection management.

  • Vulnerability Details:

    • Reentrancy Attack Potential: SWC-107: External calls in functions handling collection operations could be vulnerable.
      Severity: Medium

      (bool success, ) = collection.call{value: amount}("");

      Line: 76

    • Unchecked Arithmetic: SWC-101: Operations on collection balances may cause overflows.
      Severity: Medium

      uint256 newBalance = currentBalance - withdrawalAmount;

      Line: 89

  • Impact: Could lead to unauthorized collection manipulation, incorrect balances, or fund loss.

  • Tools Used: Manual code inspection.

  • Recommendations:

    • Reentrancy Fix: Use ReentrancyGuard to prevent reentrancy attacks.

      import "@openzeppelin/contracts/security/ReentrancyGuard.sol";
    • Arithmetic Fix: Use safe math operations to prevent overflows and underflows.

      uint256 newBalance = SafeMath.sub(currentBalance, withdrawalAmount);
Updates

Lead Judging Commences

n0kto Lead Judge 9 months ago
Submission Judgement Published
Invalidated
Reason: Too generic

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.