Zaros Part 1
Zaros
DeFiFoundry
60,000 USDC
View results
Previous Next
Submission Details
Severity: low
Invalid

Misspelled Parameter Name in `performUpkeep` Function

akay

Summary

The performUpkeep function in the LiquidationKeeper.sol contract has a misspelled parameter name. This spelling error could lead to integration issues with the Chainlink Automation system and potential contract malfunctions.

Vulnerability Details

In the LiquidationKeeper contract, the performUpkeep function is defined with a misspelled parameter name:

function performUpkeep(bytes calldata peformData) external override onlyForwarder {
// Function body
}

The parameter is spelled as peformData instead of the correct performData.

Impact

The Chainlink Automation system may fail to properly call the performUpkeep function due to the misspelled parameter, leading to the failure of automated liquidations. Also, the contract does not correctly implement the IAutomationCompatible interface, which could cause issues with contract verification and integration.

Tools Used

Manual code review

Recommendations

To fix this issue, the parameter name in the performUpkeep function should be corrected:

- function performUpkeep(bytes calldata peformData) external override onlyForwarder {
+ function performUpkeep(bytes calldata performData) external override onlyForwarder {
// Function body
}
Updates

Lead Judging Commences

inallhonesty Lead Judge
over 1 year ago
inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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

Give us feedback!