Hawk High

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

Nothing happens if the review is good, i.e. `review = true`

Summary

If the review is good, we don't change the students score.

Vulnerability Details

In LevelOne::giveReview, we update the student's score if the review is bad:

// where `false` is a bad review and true is a good review
if (!review) {
studentScore[_student] -= 10;
}

However, nothing happens if review = true. Therefore, the student's score can never be increased, even if the student deserves it.

Impact

The Student's score cannot be increased. However, not sure it's intended or not, thus, submitting as low.

Tools Used

Manual review

Recommendations

Increase the studentScore[_student]mapping value, and add a cap at 100 if needed.

Updates

Lead Judging Commences

yeahchibyke Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Design choice

Support

FAQs

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