Submission Details
Severity:
[L-03] The OwnershipTransferred event should send the oldOwner
Summary
The OwnershipTransferred event has two arparameterguments to indicate who was the current owner and who is the new owner, however parameter one does not describe the case, the parameter is declared as user when the variable should be related to the previous owner.
Impact
Confusion if the user is the owner.
Tools Used
Manual code review
Recommendations
Change the parameter name user, for example:
- event OwnershipTransferred(address indexed user, address indexed newOwner);
+ event OwnershipTransferred(address indexed oldOwner, address indexed newOwner);
Prize pool breakdown
Total prize
20,000 USDC
nSLOC
70628 USDC / LOC
18,000 USDC
2,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.