Flow

Sablier
FoundryDeFi
20,000 USDC
View results
Submission Details
Severity: low
Invalid

Contract Size Limitation

Contract: NoDelegateCall.sol

Description: The use of a private function to prevent delegate calls can increase the contract size, especially if the modifier is used multiple times throughout the contract.

  • Code Reference:

function _preventDelegateCall() private view {
if (address(this) != ORIGINAL) {
revert Errors.DelegateCall();
}
}

Impact:

  • If many functions are added to derived contracts, the size could approach the maximum limit for Solidity contracts, causing issues during deployment and limiting the ability to add future functionality.

  • Recommendations:

    • Consider alternative implementations that don't incur this penalty, such as allowing the derived contract to specify whether to enable or disable delegate calls.

    • Provide clear documentation regarding the implications of using this modifier, so developers understand the potential limitations it imposes.

Updates

Lead Judging Commences

inallhonesty Lead Judge 10 months ago
Submission Judgement Published
Invalidated
Reason: Lack of quality
inallhonesty Lead Judge 10 months ago
Submission Judgement Published
Invalidated
Reason: Lack of quality

Support

FAQs

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