Skip to content

Commit 0f8f8cd

Browse files
HaoranYiHaoranYi
HaoranYi
and
HaoranYi
authored
Revert deprecate executable feature (pyth-network#309)
* revert deprecate executable feature * add native loader account transfer test --------- Co-authored-by: HaoranYi <haoran.yi@solana.com>
1 parent aba8ce5 commit 0f8f8cd

File tree

26 files changed

+353
-729
lines changed

26 files changed

+353
-729
lines changed

‎cli/src/program.rs

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ use {
4444
},
4545
solana_rpc_client_nonce_utils::blockhash_query::BlockhashQuery,
4646
solana_sdk::{
47-
account::{is_executable, Account},
47+
account::Account,
4848
account_utils::StateMut,
4949
bpf_loader, bpf_loader_deprecated,
5050
bpf_loader_upgradeable::{self, UpgradeableLoaderState},
@@ -1066,15 +1066,6 @@ fn get_default_program_keypair(program_location: &Option<String>) -> Keypair {
10661066
program_keypair
10671067
}
10681068

1069-
fn is_account_executable(account: &Account) -> bool {
1070-
if account.owner == bpf_loader_deprecated::id() || account.owner == bpf_loader::id() {
1071-
account.executable
1072-
} else {
1073-
let feature_set = FeatureSet::all_enabled();
1074-
is_executable(account, &feature_set)
1075-
}
1076-
}
1077-
10781069
/// Deploy program using upgradeable loader. It also can process program upgrades
10791070
#[allow(clippy::too_many_arguments)]
10801071
fn process_program_deploy(
@@ -1131,7 +1122,7 @@ fn process_program_deploy(
11311122
.into());
11321123
}
11331124

1134-
if !is_account_executable(&account) {
1125+
if !account.executable {
11351126
// Continue an initial deploy
11361127
true
11371128
} else if let Ok(UpgradeableLoaderState::Program {
@@ -2534,7 +2525,7 @@ fn complete_partial_program_init(
25342525
) -> Result<(Vec<Instruction>, u64), Box<dyn std::error::Error>> {
25352526
let mut instructions: Vec<Instruction> = vec![];
25362527
let mut balance_needed = 0;
2537-
if is_account_executable(account) {
2528+
if account.executable {
25382529
return Err("Buffer account is already executable".into());
25392530
}
25402531
if account.owner != *loader_id && !system_program::check_id(&account.owner) {

‎cli/tests/program.rs

Lines changed: 12 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,9 @@ use {
1414
solana_rpc_client::rpc_client::RpcClient,
1515
solana_rpc_client_nonce_utils::blockhash_query::BlockhashQuery,
1616
solana_sdk::{
17-
account::is_executable,
1817
account_utils::StateMut,
1918
bpf_loader_upgradeable::{self, UpgradeableLoaderState},
2019
commitment_config::CommitmentConfig,
21-
feature_set::FeatureSet,
2220
pubkey::Pubkey,
2321
signature::{Keypair, NullSigner, Signer},
2422
},
@@ -102,7 +100,7 @@ fn test_cli_program_deploy_non_upgradeable() {
102100
let account0 = rpc_client.get_account(&program_id).unwrap();
103101
assert_eq!(account0.lamports, minimum_balance_for_program);
104102
assert_eq!(account0.owner, bpf_loader_upgradeable::id());
105-
assert!(is_executable(&account0, &FeatureSet::all_enabled()));
103+
assert!(account0.executable);
106104

107105
let (programdata_pubkey, _) =
108106
Pubkey::find_program_address(&[program_id.as_ref()], &bpf_loader_upgradeable::id());
@@ -112,10 +110,7 @@ fn test_cli_program_deploy_non_upgradeable() {
112110
minimum_balance_for_programdata
113111
);
114112
assert_eq!(programdata_account.owner, bpf_loader_upgradeable::id());
115-
assert!(!is_executable(
116-
&programdata_account,
117-
&FeatureSet::all_enabled()
118-
));
113+
assert!(!programdata_account.executable);
119114
assert_eq!(
120115
programdata_account.data[UpgradeableLoaderState::size_of_programdata_metadata()..],
121116
program_data[..]
@@ -143,7 +138,7 @@ fn test_cli_program_deploy_non_upgradeable() {
143138
.unwrap();
144139
assert_eq!(account1.lamports, minimum_balance_for_program);
145140
assert_eq!(account1.owner, bpf_loader_upgradeable::id());
146-
assert!(is_executable(&account1, &FeatureSet::all_enabled()));
141+
assert!(account1.executable);
147142
let (programdata_pubkey, _) = Pubkey::find_program_address(
148143
&[custom_address_keypair.pubkey().as_ref()],
149144
&bpf_loader_upgradeable::id(),
@@ -154,10 +149,7 @@ fn test_cli_program_deploy_non_upgradeable() {
154149
minimum_balance_for_programdata
155150
);
156151
assert_eq!(programdata_account.owner, bpf_loader_upgradeable::id());
157-
assert!(!is_executable(
158-
&programdata_account,
159-
&FeatureSet::all_enabled()
160-
));
152+
assert!(!programdata_account.executable);
161153
assert_eq!(
162154
programdata_account.data[UpgradeableLoaderState::size_of_programdata_metadata()..],
163155
program_data[..]
@@ -385,7 +377,7 @@ fn test_cli_program_deploy_with_authority() {
385377
let program_account = rpc_client.get_account(&program_keypair.pubkey()).unwrap();
386378
assert_eq!(program_account.lamports, minimum_balance_for_program);
387379
assert_eq!(program_account.owner, bpf_loader_upgradeable::id());
388-
assert!(is_executable(&program_account, &FeatureSet::all_enabled()));
380+
assert!(program_account.executable);
389381
let (programdata_pubkey, _) = Pubkey::find_program_address(
390382
&[program_keypair.pubkey().as_ref()],
391383
&bpf_loader_upgradeable::id(),
@@ -396,10 +388,7 @@ fn test_cli_program_deploy_with_authority() {
396388
minimum_balance_for_programdata
397389
);
398390
assert_eq!(programdata_account.owner, bpf_loader_upgradeable::id());
399-
assert!(!is_executable(
400-
&programdata_account,
401-
&FeatureSet::all_enabled()
402-
));
391+
assert!(!programdata_account.executable);
403392
assert_eq!(
404393
programdata_account.data[UpgradeableLoaderState::size_of_programdata_metadata()..],
405394
program_data[..]
@@ -433,7 +422,7 @@ fn test_cli_program_deploy_with_authority() {
433422
let program_account = rpc_client.get_account(&program_pubkey).unwrap();
434423
assert_eq!(program_account.lamports, minimum_balance_for_program);
435424
assert_eq!(program_account.owner, bpf_loader_upgradeable::id());
436-
assert!(is_executable(&program_account, &FeatureSet::all_enabled()));
425+
assert!(program_account.executable);
437426
let (programdata_pubkey, _) =
438427
Pubkey::find_program_address(&[program_pubkey.as_ref()], &bpf_loader_upgradeable::id());
439428
let programdata_account = rpc_client.get_account(&programdata_pubkey).unwrap();
@@ -442,10 +431,7 @@ fn test_cli_program_deploy_with_authority() {
442431
minimum_balance_for_programdata
443432
);
444433
assert_eq!(programdata_account.owner, bpf_loader_upgradeable::id());
445-
assert!(!is_executable(
446-
&programdata_account,
447-
&FeatureSet::all_enabled()
448-
));
434+
assert!(program_account.executable);
449435
assert_eq!(
450436
programdata_account.data[UpgradeableLoaderState::size_of_programdata_metadata()..],
451437
program_data[..]
@@ -470,7 +456,7 @@ fn test_cli_program_deploy_with_authority() {
470456
let program_account = rpc_client.get_account(&program_pubkey).unwrap();
471457
assert_eq!(program_account.lamports, minimum_balance_for_program);
472458
assert_eq!(program_account.owner, bpf_loader_upgradeable::id());
473-
assert!(is_executable(&program_account, &FeatureSet::all_enabled()));
459+
assert!(program_account.executable);
474460
let (programdata_pubkey, _) =
475461
Pubkey::find_program_address(&[program_pubkey.as_ref()], &bpf_loader_upgradeable::id());
476462
let programdata_account = rpc_client.get_account(&programdata_pubkey).unwrap();
@@ -479,10 +465,7 @@ fn test_cli_program_deploy_with_authority() {
479465
minimum_balance_for_programdata
480466
);
481467
assert_eq!(programdata_account.owner, bpf_loader_upgradeable::id());
482-
assert!(!is_executable(
483-
&programdata_account,
484-
&FeatureSet::all_enabled()
485-
));
468+
assert!(program_account.executable);
486469
assert_eq!(
487470
programdata_account.data[UpgradeableLoaderState::size_of_programdata_metadata()..],
488471
program_data[..]
@@ -548,7 +531,7 @@ fn test_cli_program_deploy_with_authority() {
548531
let program_account = rpc_client.get_account(&program_pubkey).unwrap();
549532
assert_eq!(program_account.lamports, minimum_balance_for_program);
550533
assert_eq!(program_account.owner, bpf_loader_upgradeable::id());
551-
assert!(is_executable(&program_account, &FeatureSet::all_enabled()));
534+
assert!(program_account.executable);
552535
let (programdata_pubkey, _) =
553536
Pubkey::find_program_address(&[program_pubkey.as_ref()], &bpf_loader_upgradeable::id());
554537
let programdata_account = rpc_client.get_account(&programdata_pubkey).unwrap();
@@ -557,10 +540,7 @@ fn test_cli_program_deploy_with_authority() {
557540
minimum_balance_for_programdata
558541
);
559542
assert_eq!(programdata_account.owner, bpf_loader_upgradeable::id());
560-
assert!(!is_executable(
561-
&programdata_account,
562-
&FeatureSet::all_enabled()
563-
));
543+
assert!(program_account.executable);
564544
assert_eq!(
565545
programdata_account.data[UpgradeableLoaderState::size_of_programdata_metadata()..],
566546
program_data[..]

‎ledger-tool/src/program.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -540,7 +540,6 @@ pub fn program(ledger_path: &Path, matches: &ArgMatches<'_>) {
540540
.get_current_instruction_context()
541541
.unwrap(),
542542
true, // copy_account_data
543-
&invoke_context.feature_set,
544543
)
545544
.unwrap();
546545

‎program-runtime/src/invoke_context.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -403,7 +403,7 @@ impl<'a> InvokeContext<'a> {
403403
})?;
404404
let borrowed_program_account = instruction_context
405405
.try_borrow_instruction_account(self.transaction_context, program_account_index)?;
406-
if !borrowed_program_account.is_executable(&self.feature_set) {
406+
if !borrowed_program_account.is_executable() {
407407
ic_msg!(self, "Account {} is not executable", callee_program_id);
408408
return Err(InstructionError::AccountNotExecutable);
409409
}
@@ -802,17 +802,17 @@ mod tests {
802802
MockInstruction::NoopFail => return Err(InstructionError::GenericError),
803803
MockInstruction::ModifyOwned => instruction_context
804804
.try_borrow_instruction_account(transaction_context, 0)?
805-
.set_data_from_slice(&[1], &invoke_context.feature_set)?,
805+
.set_data_from_slice(&[1])?,
806806
MockInstruction::ModifyNotOwned => instruction_context
807807
.try_borrow_instruction_account(transaction_context, 1)?
808-
.set_data_from_slice(&[1], &invoke_context.feature_set)?,
808+
.set_data_from_slice(&[1])?,
809809
MockInstruction::ModifyReadonly => instruction_context
810810
.try_borrow_instruction_account(transaction_context, 2)?
811-
.set_data_from_slice(&[1], &invoke_context.feature_set)?,
811+
.set_data_from_slice(&[1])?,
812812
MockInstruction::UnbalancedPush => {
813813
instruction_context
814814
.try_borrow_instruction_account(transaction_context, 0)?
815-
.checked_add_lamports(1, &invoke_context.feature_set)?;
815+
.checked_add_lamports(1)?;
816816
let program_id = *transaction_context.get_key_of_account_at_index(3)?;
817817
let metas = vec![
818818
AccountMeta::new_readonly(
@@ -843,7 +843,7 @@ mod tests {
843843
}
844844
MockInstruction::UnbalancedPop => instruction_context
845845
.try_borrow_instruction_account(transaction_context, 0)?
846-
.checked_add_lamports(1, &invoke_context.feature_set)?,
846+
.checked_add_lamports(1)?,
847847
MockInstruction::ConsumeComputeUnits {
848848
compute_units_to_consume,
849849
desired_result,
@@ -855,7 +855,7 @@ mod tests {
855855
}
856856
MockInstruction::Resize { new_len } => instruction_context
857857
.try_borrow_instruction_account(transaction_context, 0)?
858-
.set_data(vec![0; new_len as usize], &invoke_context.feature_set)?,
858+
.set_data(vec![0; new_len as usize])?,
859859
}
860860
} else {
861861
return Err(InstructionError::InvalidInstructionData);

‎program-runtime/src/message_processor.rs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -221,16 +221,16 @@ mod tests {
221221
MockSystemInstruction::TransferLamports { lamports } => {
222222
instruction_context
223223
.try_borrow_instruction_account(transaction_context, 0)?
224-
.checked_sub_lamports(lamports, &invoke_context.feature_set)?;
224+
.checked_sub_lamports(lamports)?;
225225
instruction_context
226226
.try_borrow_instruction_account(transaction_context, 1)?
227-
.checked_add_lamports(lamports, &invoke_context.feature_set)?;
227+
.checked_add_lamports(lamports)?;
228228
Ok(())
229229
}
230230
MockSystemInstruction::ChangeData { data } => {
231231
instruction_context
232232
.try_borrow_instruction_account(transaction_context, 1)?
233-
.set_data(vec![data], &invoke_context.feature_set)?;
233+
.set_data(vec![data])?;
234234
Ok(())
235235
}
236236
}
@@ -444,14 +444,14 @@ mod tests {
444444
MockSystemInstruction::DoWork { lamports, data } => {
445445
let mut dup_account = instruction_context
446446
.try_borrow_instruction_account(transaction_context, 2)?;
447-
dup_account.checked_sub_lamports(lamports, &invoke_context.feature_set)?;
448-
to_account.checked_add_lamports(lamports, &invoke_context.feature_set)?;
449-
dup_account.set_data(vec![data], &invoke_context.feature_set)?;
447+
dup_account.checked_sub_lamports(lamports)?;
448+
to_account.checked_add_lamports(lamports)?;
449+
dup_account.set_data(vec![data])?;
450450
drop(dup_account);
451451
let mut from_account = instruction_context
452452
.try_borrow_instruction_account(transaction_context, 0)?;
453-
from_account.checked_sub_lamports(lamports, &invoke_context.feature_set)?;
454-
to_account.checked_add_lamports(lamports, &invoke_context.feature_set)?;
453+
from_account.checked_sub_lamports(lamports)?;
454+
to_account.checked_add_lamports(lamports)?;
455455
Ok(())
456456
}
457457
}

‎program-test/src/lib.rs

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,6 @@ pub fn invoke_builtin_function(
133133
.transaction_context
134134
.get_current_instruction_context()?,
135135
true, // copy_account_data // There is no VM so direct mapping can not be implemented here
136-
&invoke_context.feature_set,
137136
)?;
138137

139138
// Deserialize data back into instruction params
@@ -164,25 +163,18 @@ pub fn invoke_builtin_function(
164163
if borrowed_account.is_writable() {
165164
if let Some(account_info) = account_info_map.get(borrowed_account.get_key()) {
166165
if borrowed_account.get_lamports() != account_info.lamports() {
167-
borrowed_account
168-
.set_lamports(account_info.lamports(), &invoke_context.feature_set)?;
166+
borrowed_account.set_lamports(account_info.lamports())?;
169167
}
170168

171169
if borrowed_account
172170
.can_data_be_resized(account_info.data_len())
173171
.is_ok()
174-
&& borrowed_account
175-
.can_data_be_changed(&invoke_context.feature_set)
176-
.is_ok()
172+
&& borrowed_account.can_data_be_changed().is_ok()
177173
{
178-
borrowed_account.set_data_from_slice(
179-
&account_info.data.borrow(),
180-
&invoke_context.feature_set,
181-
)?;
174+
borrowed_account.set_data_from_slice(&account_info.data.borrow())?;
182175
}
183176
if borrowed_account.get_owner() != account_info.owner {
184-
borrowed_account
185-
.set_owner(account_info.owner.as_ref(), &invoke_context.feature_set)?;
177+
borrowed_account.set_owner(account_info.owner.as_ref())?;
186178
}
187179
}
188180
}
@@ -293,17 +285,17 @@ impl solana_sdk::program_stubs::SyscallStubs for SyscallStubs {
293285
.unwrap();
294286
if borrowed_account.get_lamports() != account_info.lamports() {
295287
borrowed_account
296-
.set_lamports(account_info.lamports(), &invoke_context.feature_set)
288+
.set_lamports(account_info.lamports())
297289
.unwrap();
298290
}
299291
let account_info_data = account_info.try_borrow_data().unwrap();
300292
// The redundant check helps to avoid the expensive data comparison if we can
301293
match borrowed_account
302294
.can_data_be_resized(account_info_data.len())
303-
.and_then(|_| borrowed_account.can_data_be_changed(&invoke_context.feature_set))
295+
.and_then(|_| borrowed_account.can_data_be_changed())
304296
{
305297
Ok(()) => borrowed_account
306-
.set_data_from_slice(&account_info_data, &invoke_context.feature_set)
298+
.set_data_from_slice(&account_info_data)
307299
.unwrap(),
308300
Err(err) if borrowed_account.get_data() != *account_info_data => {
309301
panic!("{err:?}");
@@ -313,7 +305,7 @@ impl solana_sdk::program_stubs::SyscallStubs for SyscallStubs {
313305
// Change the owner at the end so that we are allowed to change the lamports and data before
314306
if borrowed_account.get_owner() != account_info.owner {
315307
borrowed_account
316-
.set_owner(account_info.owner.as_ref(), &invoke_context.feature_set)
308+
.set_owner(account_info.owner.as_ref())
317309
.unwrap();
318310
}
319311
if instruction_account.is_writable {

0 commit comments

Comments
 (0)