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 9 months ago
Submission Judgement Published
Invalidated
Reason: Lack of quality
inallhonesty Lead Judge 9 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.