Hawk High

First Flight #39
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Severity: high
Valid

System Upgrade Proceeds Without Valid Session End or Complete Reviews

Summary

The system currently allows graduateAndUpgrade() to proceed without verifying that each student has received all 4 reviews. This breaks the integrity of the school lifecycle, where graduation and upgrades should only occur after full evaluation.

Vulnerability Details

https://github.com/CodeHawks-Contests/2025-05-hawk-high/blob/3a7251910c31739505a8699c7a0fc1b7de2c30b5/src\LevelOne.sol#L295-L295

Impact

Students may graduate without proper evaluation.

Tools Used

manual review

Recommendations

Add a loop to verify all students have received 4 reviews:

for (uint256 i = 0; i < listOfStudents.length; i++) {
if (studentReviews[listOfStudents[i]].length != 4) {
revert IncompleteStudentReviews(listOfStudents[i]);
}
}



Updates

Lead Judging Commences

yeahchibyke Lead Judge 6 months ago
Submission Judgement Published
Validated
Assigned finding tags:

cut-off criteria not applied

All students are graduated when the graduation function is called as the cut-off criteria is not applied.

Support

FAQs

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