Submission Details
Severity:
Missing Refund Deadline Edge Case in RustFund Contract
Summary
The refund function doesn’t handle deadline = 0, potentially allowing refunds before a deadline is set.
Vulnerability Details:
The vulnerable code is in the refund function:
rust
if ctx.accounts.fund.deadline != 0 && ctx.accounts.fund.deadline > Clock::get().unwrap().unix_timestamp.try_into().unwrap() {
return Err(ErrorCode::DeadlineNotReached.into());
}
No Check: deadline = 0 skips the condition, proceeding to refund.
Impact:
Funds Misallocation: Unintended refunds in uninitialized campaigns.
Tools Used: Manual Review
Recommendations:
Add check:
rust
if fund.deadline == 0 {
return Err(ErrorCode::DeadlineNotSet.into());
}
Add new error code:
rust
#[error_code]
pub enum ErrorCode {
// ... existing errors ...
#[msg("Deadline not set")]
DeadlineNotSet,
}
Updates
Appeal created
bube 2 months ago
Submission Judgement Published Assigned finding tags:
Possible refund before the deadline is initialized
Rewards breakdown
nSLOC
170This 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.