RustFund

First Flight #36
Beginner FriendlyRust
100 EXP
View results
Submission Details
Severity: medium
Valid

Fund Deadline Can Be Set Multiple Times

Summary

Fund Deadline Can Be Set Multiple Times

Vulnerability Details

The dealine_set flag is not updated after setting the deadline, meaning it can be set multiple times.

pub fn set_deadline(ctx: Context<FundSetDeadline>, deadline: u64) -> Result<()> {
let fund = &mut ctx.accounts.fund;
if fund.dealine_set {
return Err(ErrorCode::DeadlineAlreadySet.into());
}
fund.deadline = deadline;
Ok(())
}

Impact

Fund Deadline Can Be Set Multiple Times

Tools Used

Manual review.

Recommendations

Set fund.dealine_set = true after setting the deadline.

Updates

Appeal created

bube Lead Judge about 2 months ago
Submission Judgement Published
Validated
Assigned finding tags:

Deadline set flag is not updated in `set_deadline` function

Support

FAQs

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