Audit · firm’s term: audit report
QuillAudits reviewed OneWave on 28 Apr 2025: 2 findings, 0 critical; none marked fixed.
Report hosted on quillaudits.com.
Trustblock does not review code.
Statuses are the security firm’s own, from its report.
| Severity | Open | Acknowledged | Fixed |
|---|---|---|---|
| Critical | 0 | 0 | 0 |
| High | 0 | 0 | 0 |
| Medium | 0 | 0 | 0 |
| Low | 2 | 0 | 0 |
The issue reported is a variable/function shadowing problem. In the <code>WAVE</code> contract, the <code>_approve()</code> function has a parameter named <code>owner</code>, which shadows the <code>owner()</code> function from the <code>Ownable</code> contract. This can lead to confusion and potential errors in the code, as the <code>owner</code> parameter in <code>_approve</code> is a local variable and not the <code>owner()</code> function that returns the contract owner address.
The contract declares that it implements the ERC721 interface, but no functions or functionalities from this interface are utilized within the contract's code. This unnecessary declaration increases the contract's size and could lead to potential confusion regarding its intended behavior. Removing this unused interface will optimize the contract's bytecode and improve its clarity.
The report names a repository or files. Open the report to see its scope.
350 projects reviewed · 459 reports · latest Apr 2025
See QuillAudits’s auditsSomething wrong on this page? A missing report, a wrong date, a finding that was fixed since. Tell us and we check it against its source.
Tell us