Sparkn

CodeFox Inc.
DeFiFoundryProxy
15,000 USDC
View results
Submission Details
Severity: medium

Incorrect Error naming in _distribute

Vulnerability Details

The proxyFactory#_distribute method calls the proxy with data as params.
It then checks for the call success using the returned variable success.
But it emits the wrong event.
This is a static call not a Delegate call.

function _distribute(address proxy, bytes calldata data) internal {
(bool success,) = proxy.call(data);
if (!success) revert ProxyFactory__DelegateCallFailed();
..
}

Impact

  • Misleading for the users interacting with the protocol

  • Loss of user's confidence on platform

Tools Used

Manual review

Recommendations

Rename the error to ProxyFactory__DistributionCallFailed()

Support

FAQs

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