Submission Details
Severity:
Event should have old and new value as parameters.
Summary
Events emitted when state is change, should contain both the old and the new value.
Tools Used
Manual Review
Recommendations
Do the following:
- event FeeAddressChanged(address newFeeAddress);
+ event FeeAddressChanged(address oldFeeAddress, address newFeeAddress);
- feeAddress = newFeeAddress;
- emit FeeAddressChanged(newFeeAddress);
+ address oldFeeAddress = feeAddress;
+ feeAddress = newFeeAddress;
+ emit FeeAddressChanged(oldFeeAddress, newFeeAddress);
Updates
Lead Judging Commences
Hamiltonite over 1 year ago
Submission Judgement Published Assigned finding tags:
events are missing or not detailed enough
Rewards breakdown
nSLOC
143This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
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.