Skip to content

Commit

Permalink
sdk: add Immutable and IncorrectAuthority to ProgramError (#35113)
Browse files Browse the repository at this point in the history
  • Loading branch information
Joe C authored Feb 7, 2024
1 parent bc735fa commit 3ddd235
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions sdk/program/src/program_error.rs
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,10 @@ pub enum ProgramError {
InvalidAccountOwner,
#[error("Program arithmetic overflowed")]
ArithmeticOverflow,
#[error("Account is immutable")]
Immutable,
#[error("Incorrect authority provided")]
IncorrectAuthority,
}

pub trait PrintProgramError {
Expand Down Expand Up @@ -113,6 +117,8 @@ impl PrintProgramError for ProgramError {
}
Self::InvalidAccountOwner => msg!("Error: InvalidAccountOwner"),
Self::ArithmeticOverflow => msg!("Error: ArithmeticOverflow"),
Self::Immutable => msg!("Error: Immutable"),
Self::IncorrectAuthority => msg!("Error: IncorrectAuthority"),
}
}
}
Expand Down Expand Up @@ -149,6 +155,8 @@ pub const MAX_INSTRUCTION_TRACE_LENGTH_EXCEEDED: u64 = to_builtin!(21);
pub const BUILTIN_PROGRAMS_MUST_CONSUME_COMPUTE_UNITS: u64 = to_builtin!(22);
pub const INVALID_ACCOUNT_OWNER: u64 = to_builtin!(23);
pub const ARITHMETIC_OVERFLOW: u64 = to_builtin!(24);
pub const IMMUTABLE: u64 = to_builtin!(25);
pub const INCORRECT_AUTHORITY: u64 = to_builtin!(26);
// Warning: Any new program errors added here must also be:
// - Added to the below conversions
// - Added as an equivalent to InstructionError
Expand Down Expand Up @@ -187,6 +195,8 @@ impl From<ProgramError> for u64 {
}
ProgramError::InvalidAccountOwner => INVALID_ACCOUNT_OWNER,
ProgramError::ArithmeticOverflow => ARITHMETIC_OVERFLOW,
ProgramError::Immutable => IMMUTABLE,
ProgramError::IncorrectAuthority => INCORRECT_AUTHORITY,
ProgramError::Custom(error) => {
if error == 0 {
CUSTOM_ZERO
Expand Down Expand Up @@ -227,6 +237,8 @@ impl From<u64> for ProgramError {
}
INVALID_ACCOUNT_OWNER => Self::InvalidAccountOwner,
ARITHMETIC_OVERFLOW => Self::ArithmeticOverflow,
IMMUTABLE => Self::Immutable,
INCORRECT_AUTHORITY => Self::IncorrectAuthority,
_ => Self::Custom(error as u32),
}
}
Expand Down Expand Up @@ -267,6 +279,8 @@ impl TryFrom<InstructionError> for ProgramError {
}
Self::Error::InvalidAccountOwner => Ok(Self::InvalidAccountOwner),
Self::Error::ArithmeticOverflow => Ok(Self::ArithmeticOverflow),
Self::Error::Immutable => Ok(Self::Immutable),
Self::Error::IncorrectAuthority => Ok(Self::IncorrectAuthority),
_ => Err(error),
}
}
Expand Down Expand Up @@ -305,6 +319,8 @@ where
}
INVALID_ACCOUNT_OWNER => Self::InvalidAccountOwner,
ARITHMETIC_OVERFLOW => Self::ArithmeticOverflow,
IMMUTABLE => Self::Immutable,
INCORRECT_AUTHORITY => Self::IncorrectAuthority,
_ => {
// A valid custom error has no bits set in the upper 32
if error >> BUILTIN_BIT_SHIFT == 0 {
Expand Down

0 comments on commit 3ddd235

Please sign in to comment.