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 #662
Labels
bug
Something isn't working
grade-b
Q-12
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
Missing event emission and lack of parameters validations in setters1R 1NC Deprecated LibraryR Emitted amount can be bigger than expectedUnclear, skipping (also unprofessional notes) _safeMint() should be used rather than _mint() wherever possibleL Single-step process for critical ownership transfer/renounce is riskyNC Missing checks for address(0x0) when assigning values to address state or immutable variablesL 2L 2R 2NC |
GainsGoblin marked the issue as sponsor confirmed |
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
C4-Staff
added a commit
that referenced
this issue
Jan 6, 2023
This was referenced Jan 15, 2023
3L from dups 5L 2R 2Nc |
GalloDaSballo marked the issue as grade-b |
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-12
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: