Part 2
Zaros
PerpetualsDEXFoundrySolidity
70,000 USDC
View results
Previous Next
Submission Details
Severity: low
Invalid

Insufficient Event Emission

luffy

Summary

Critical parameter changes don't emit events, hindering off-chain monitoring.

Vulnerability Details

function setAutoDeleveragingParams(
uint256 threshold,
uint256 factor,
uint256 scale
) external onlyAdmin {
// No event emitted
autoDeleveragingThreshold = threshold;
autoDeleveragingFactor = factor;
autoDeleveragingScale = scale;
}

Impact

  • Difficult audit trail for parameter changes

  • Reduced protocol transparency

Tools Used

  • Manual review

  • Slither

Recommendations

event AutoDeleveragingParamsUpdated(
uint256 threshold,
uint256 factor,
uint256 scale
);
​
function setAutoDeleveragingParams(
uint256 threshold,
uint256 factor,
uint256 scale
) external onlyAdmin {
autoDeleveragingThreshold = threshold;
autoDeleveragingFactor = factor;
autoDeleveragingScale = scale;
emit AutoDeleveragingParamsUpdated(threshold, factor, scale);
}
Updates

Lead Judging Commences

inallhonesty Lead Judge 4 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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