New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
QA Report #658
Labels
bug
Something isn't working
grade-b
Q-11
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
Comments
code423n4
added
bug
Something isn't working
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
labels
Dec 16, 2022
code423n4
added a commit
that referenced
this issue
Dec 16, 2022
[L-01] NO UPPER LIMIT ON NUMBER OF REWARD ASSET TOKENS IN GovNFT CONTRACTL [L-02] super._safeMint FUNCTION CAN BE CALLED INSTEAD OF super._mint FUNCTION FOR MINTING GOVERNANCE NFTSL [L-03] super._safeTransfer FUNCTION CAN BE CALLED INSTEAD OF super._transfer FUNCTION FOR TRANSFERRING GOVERNANCE NFTSL [L-04] USDT IS NOT SUPPORTED FOR CALLING Trading._handleDeposit FUNCTION ON ETHEREUM MAINNETDup 198 / M-12 [L-05] MARGIN ASSET TOKENS WITH MORE THAN 18 DECIMALS ARE NOT SUPPORTEDDup 533 / M-01 [L-06] UNSAFE decimals() CALL FOR MARGIN ASSET TOKENS THAT DO NOT IMPLEMENT decimals()L |
c4-sponsor
added
the
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
label
Jan 5, 2023
GainsGoblin marked the issue as sponsor confirmed |
C4-Staff
added a commit
that referenced
this issue
Jan 6, 2023
4L |
c4-judge
added a commit
that referenced
this issue
Jan 22, 2023
c4-judge
added a commit
that referenced
this issue
Jan 22, 2023
2L from dups 6L |
GalloDaSballo marked the issue as grade-b |
Despite the report "mathematically not making it", I chose to award it a B because it in aggregate is very valuable (6 Low Severity findings) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
Something isn't working
grade-b
Q-11
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
sponsor confirmed
Sponsor agrees this is a problem and intends to fix it (OK to use w/ "disagree with severity")
See the markdown file with the details of this report here.
The text was updated successfully, but these errors were encountered: