fix: add a funding account to the resizemapping ix

Author: tejasbadadare

program/rust/src/instruction.rs

@@ -109,6 +109,7 @@ pub enum OracleCommand {
     // account[1] price account          [writable]
     // account[2] permissions account    [writable]
     InitPriceFeedIndex    = 19,
+    /// Resize mapping account
     // account[0] funding account [signer writable]
     // account[1] mapping account [writable]
     ResizeMapping         = 20,

program/rust/src/processor/resize_mapping.rs

@@ -32,19 +32,26 @@ use {
 };
 
 /// Resize mapping account.
-// account[1] mapping account [writable]
+/// account[0] funding account [signer writable]
+/// account[1] mapping account [writable]
 pub fn resize_mapping(
     program_id: &Pubkey,
     accounts: &[AccountInfo],
     instruction_data: &[u8],
 ) -> ProgramResult {
-    let mapping_account = match accounts {
-        [x] => Ok(x),
-        _ => Err(OracleError::InvalidNumberOfAccounts),
-    }?;
+    let (funding_account, mapping_account) = match accounts {
+        [funding, mapping] => (funding, mapping),
+        _ => return Err(OracleError::InvalidNumberOfAccounts.into()),
+    };
+
+    // Verify funding account is a signer
+    if !funding_account.is_signer {
+        return Err(solana_program::program_error::ProgramError::MissingRequiredSignature);
+    }
 
     let hdr = load::<CommandHeader>(instruction_data)?;
 
+    // Verify mapping account is writable and owned by program
     check_valid_writable_account(program_id, mapping_account)?;
 
     {

program/rust/src/tests/pyth_simulator.rs

@@ -492,12 +492,15 @@ impl PythSimulator {
         let instruction = Instruction::new_with_bytes(
             self.program_id,
             bytes_of(&cmd),
-            vec![AccountMeta::new(mapping_keypair.pubkey(), true)],
+            vec![
+                AccountMeta::new(self.genesis_keypair.pubkey(), true), // funding account
+                AccountMeta::new(mapping_keypair.pubkey(), false),     // mapping account
+            ],
         );
 
         self.process_ixs(
             &[instruction],
-            &vec![mapping_keypair],
+            &vec![],
             &copy_keypair(&self.genesis_keypair),
         )
         .await

program/rust/src/tests/test_resize_mapping.rs

@@ -17,6 +17,8 @@ use {
 async fn test_resize_mapping() {
     let mut sim = PythSimulator::new().await;
     let mapping_keypair = sim.init_mapping().await.unwrap();
+
+    // First attempt should fail since account doesn't need resizing
     assert_eq!(
         sim.resize_mapping(&mapping_keypair)
             .await
@@ -28,6 +30,7 @@ async fn test_resize_mapping() {
         )
     );
 
+    // Truncate account to old size to test resizing
     sim.truncate_account(mapping_keypair.pubkey(), 20536).await; // Old size.
     for i in 0..14 {
         println!("resize #{i}");