Incorrect comments #49
Labels
1 (Low Risk)
Assets are not at risk. State handling, function incorrect as to spec, issues with comments
bug
Something isn't working
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
Handle
ye0lde
Vulnerability details
Impact
Code clarity
Proof of Concept
"@param" should be "@return"
https://github.com/code-423n4/2021-11-overlay/blob/1833b792caf3eb8756b1ba5f50f9c2ce085e54d0/contracts/market/OverlayV1Market.sol#L83-L85
Not sure what this comment is for maybe just needs to be deleted.
https://github.com/code-423n4/2021-11-overlay/blob/1833b792caf3eb8756b1ba5f50f9c2ce085e54d0/contracts/mothership/OverlayV1Mothership.sol#L155
Tools Used
Visual Studio Code, Remix
Recommended Mitigation Steps
Correct the comments if the suggestions are valid.
The text was updated successfully, but these errors were encountered: