Changed inner_test_validate_owner to return Ok(())

dkcumming · dkcumming · commit ef55bb57e64c · 2025-11-30T01:05:15.000+10:00
Previously in the happy case `result` was returned, but that may be a
different error unrelated to signers which will then get propagated by
`inner_test_validate_owner`.
diff --git a/p-token/src/entrypoint-runtime-verification.rs b/p-token/src/entrypoint-runtime-verification.rs
@@ -750,9 +750,9 @@ fn inner_test_validate_owner(
             if signers_count < multisig.m as usize {
                 assert_eq!(result, Err(ProgramError::MissingRequiredSignature));
                 return result;
-            } else {
-                return result;
             }
+
+            Ok(())
         }
     }
     // Line 133-135: Non-multisig case - check if owner_account_info.is_signer()

Original file line number	Diff line number	Diff line change
`@@ -750,9 +750,9 @@ fn inner_test_validate_owner(`
`750`	`750`	`if signers_count < multisig.m as usize {`
`751`	`751`	`assert_eq!(result, Err(ProgramError::MissingRequiredSignature));`
`752`	`752`	`return result;`
`753`		`- } else {`
`754`		`- return result;`
`755`	`753`	`}`
	`754`	`+`
	`755`	`+ Ok(())`
`756`	`756`	`}`
`757`	`757`	`}`
`758`	`758`	`// Line 133-135: Non-multisig case - check if owner_account_info.is_signer()`