Liquidity Management
Gamma
DeFiFoundry
50,000 USDC
View results
Previous Next
Submission Details
Severity: low
Invalid

important functions should emit events.

mizila_firox

Summary

it's important for transparency to emit events for these kinds of changes.

Vulnerability Details

https://github.com/CodeHawks-Contests/2025-02-gamma/blob/main/contracts/PerpetualVault.sol

function setKeeper(address _keeper) external onlyOwner {
if (_keeper == address(0)) revert Error.ZeroValue();
keeper = _keeper;
}
function setTreasury(address _treasury) external onlyOwner {
if (_treasury == address(0)) revert Error.ZeroValue();
treasury = _treasury;
}
function setVaultReader(address _vaultReader) external onlyOwner {
vaultReader = IVaultReader(_vaultReader);
}
function setDepositPaused(bool _depositPaused) external onlyOwner {
depositPaused = _depositPaused;
}

Impact

important changes like these should emit events for transparency.

Tools Used

manual

Recommendations

implement events to each of them

Updates

Lead Judging Commences

n0kto Lead Judge 8 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Informational or Gas

Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelihood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.

n0kto Lead Judge 8 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Informational or Gas

Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelihood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.

Support

FAQs

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