Submission Details
Impact:
Likelihood:
[EVMN-HH07] Improper Error Handling in `expel()` Function
Summary
Improper Error Handling in expel() Function
Vulnerability Details
The expel function uses a generic revert() without a message or custom error when checking if the school is in session:
function expel(address \_student) public onlyPrincipal {
if (inSession == false) {
revert();
}
// ...
}
Impact
Low
Tools Used
Manual review.
Recommendations
Use a proper custom error for better error handling and gas efficiency:
error HH__NotInSession();
function expel(address _student) public onlyPrincipal {
if (inSession == false) {
revert HH__NotInSession();
}
// ...
}
Rewards breakdown
nSLOC
243This 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.