Hawk High

First Flight #39
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Impact: low
Likelihood: low
Invalid

[L-01] Incorrect `revert()` in `expel`

Summary

In expel, when if (inSession == false) is true, it directly calls revert(). Since a bare revert() does not provide specific error information, the caller may not understand the reason for the failure.

Vulnerability Details

Impact

The lack of specific error information means that the cause of the failure cannot be directly determined when used.

POC

Not written.

Recommendations

+ error HH__NotInSession();
if (inSession == false) {
+ revert HH__NotInSession();
}
Updates

Lead Judging Commences

yeahchibyke Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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