Project
One World
NFTDeFi
15,000 USDC
View results
Previous Next
Submission Details
Severity: low
Invalid

Typo in `EIP712Base::getDomainSeparator`

y4y

Summary

In the contract, there are multiple typos around the word separator, if the contract is meant to strictly comply with EIP712, it can cause the intended getDomainSeparator function not behave intended because of the typo.

Vulnerability Details

For example, here:

bytes32 internal domainSeperator;

Which the spelling is Seperator instead of Separator.

Impact

Function calls to getDomainSeparator would fail, depending on wether the contract is meant to comply strictly with EIP712, it ranges from Low to Medium.

Tools Used

Manual review.

Recommendations

Change "seperator" to "separator"

Updates

Lead Judging Commences

0xbrivan2 Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
0xbrivan2 Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.

Give us feedback!