code-423n4 / 2022-12-tigris-findings

8 stars 4 forks source link

QA Report #662

Open code423n4 opened 1 year ago

code423n4 commented 1 year ago

See the markdown file with the details of this report here.

GalloDaSballo commented 1 year ago

Missing event emission and lack of parameters validations in setters

1R 1NC

Deprecated Library

R

Emitted amount can be bigger than expected

Unclear, skipping (also unprofessional notes)

_safeMint() should be used rather than _mint() wherever possible

L

Single-step process for critical ownership transfer/renounce is risky

NC

Missing checks for address(0x0) when assigning values to address state or immutable variables

L

2L 2R 2NC

c4-sponsor commented 1 year ago

GainsGoblin marked the issue as sponsor confirmed

GalloDaSballo commented 1 year ago

3L from dups

5L 2R 2Nc

c4-judge commented 1 year ago

GalloDaSballo marked the issue as grade-b