DeFiFoundry
60,000 USDC
View results
Submission Details
Severity: low
Invalid

depositMargin function visibility should be external

Summary

https://github.com/Cyfrin/2024-07-zaros/blob/main/src/perpetuals/branches/TradingAccountBranch.sol#L317

function depositMargin(uint128 tradingAccountId, address collateralType, uint256 amount) public virtual

depositMargin is not called by TradingAccountBranch itself, there is no need for public visibility

Updates

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Lack of quality

Support

FAQs

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

Give us feedback!