Submission Details
Severity:
Wrong nonReentrant modifier
Summary
nonReentrant modifier lacks an important assignment.
Vulnerability Details
With the current implementation the variable lockedstays always false and thus it will pass the require check leading to reentrancy.
Impact
All funcitons that has nonReentrant modifier will be vulnerable to reentrancy attacks
Tools Used
Recommendations
The modifier should set locked to true before the _ keyword.
modifier nonReentrant() {
require(!locked, "No re-entrancy");
+ locked = true;
_;
locked = false;
}
Updates
Lead Judging Commences
0xtimefliez 11 months ago
Submission Judgement Published Assigned finding tags:
mutex lock incomplete
Rewards breakdown
nSLOC
129This 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.