DeFiFoundry
50,000 USDC
View results
Submission Details
Severity: high
Invalid

Missing Access Control in KeeperProxy.sol

Summary

The KeeperProxy.sol contract lacks proper access control for certain functions, allowing unauthorized users to execute critical operations

Vulnerability Details

The cancelFlow function does not have an onlyKeeper modifier, allowing anyone to cancel ongoing flows

function cancelFlow() external {
// Cancel flow logic
}

Impact

An attacker could disrupt the protocol by canceling ongoing flows, leading to loss of funds or failed trades

Tools Used

Manual code review

Recommendations

Add the onlyKeeper modifier to the cancelFlow function

function cancelFlow() external onlyKeeper {
// Cancel flow logic
}
Updates

Lead Judging Commences

n0kto Lead Judge 3 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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