Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Solana - Low & Informational #230

Merged
merged 15 commits into from
Jan 10, 2025
Merged

Solana - Low & Informational #230

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
f2afb4e
1: Fix copypasta
djb15 Jan 9, 2025
b5c5aa6
2: Set a hard cap on checkpoint account limit
djb15 Jan 9, 2025
697d6e4
5: Remove redundant PDA checks in withdraw
djb15 Jan 9, 2025
d7c940e
8: Use element size constants
djb15 Jan 10, 2025
2041476
9: Prevent 0 withdrawals
djb15 Jan 10, 2025
9ca777a
13: Claim vest refunds vesting admin
djb15 Jan 10, 2025
28afa60
14: Increase maximum number of checkpoint accounts
djb15 Jan 10, 2025
0954e68
12: Remove unnecessary current delegate check
djb15 Jan 10, 2025
b6289bc
16: Allow signer to cap lamports when executing arbitrary instructions
djb15 Jan 10, 2025
11e3053
11: Let governance control hub proposal metadata
djb15 Jan 10, 2025
e0b191a
15: Self calls use special airlock
djb15 Jan 10, 2025
5d3d9e0
Check owner of payer after cpis
djb15 Jan 10, 2025
3d8e978
Fix constraint after rebase
djb15 Jan 10, 2025
66b5cd9
Fix new test to use proper seed
djb15 Jan 10, 2025
c1ed1f3
Comment flaky test
djb15 Jan 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions solana/programs/staking/src/error.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -173,4 +173,6 @@ pub enum MessageExecutorError {
InvalidWormholeChainId,
#[msg("The executed instructions exceeded max lamports")]
ExceededMaxLamports,
#[msg("The account owner of the signer was changed")]
SignerAccountOwernshipChanged,
}
5 changes: 5 additions & 0 deletions solana/programs/staking/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -774,6 +774,11 @@ pub mod staking {
MessageExecutorError::ExceededMaxLamports
);

require!(
ctx.accounts.payer.owner.key() == ctx.accounts.system_program.key(),
MessageExecutorError::SignerAccountOwernshipChanged
);

Ok(())
}

Expand Down
Loading