Submission Details
Severity:
Direct Lamport Transfer Instead of CPI Transfer
Summary
The function uses direct lamport borrowing for balance transfers, bypassing Solana’s CPI (Cross-Program Invocation) mechanism, which is a secure standard for token transfers.
Vulnerability Details
Direct lamport borrowing does not enforce system program constraints.
May cause issues with rent exemption and account validity.
Impact
Possible inconsistencies in account balances.
Increased vulnerability to mismanagement or unintended behavior in account balances.
Tools Used
Manual Code Review
Solana Docs Reference
Recommendations
Use a CPI transfer instead of direct lamport borrowing:
invoke(
&system_instruction::transfer(
ctx.accounts.fund.to_account_info().key,
ctx.accounts.creator.to_account_info().key,
amount,
),
&[
ctx.accounts.fund.to_account_info(),
ctx.accounts.creator.to_account_info(),
ctx.accounts.system_program.to_account_info(),
],
)?;
Updates
Appeal created
bube 5 months ago
Submission Judgement Published Assigned finding tags:
Unsafe direct lamport manipulation
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.